This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch: Remove non-mapped location support


>>>>> "Manuel" == Manuel LÃpez-IbÃÃez <lopezibanez@gmail.com> writes:

Manuel> But that can be factor out in a function.

Yeah, true.

Manuel> My point is that having
Manuel>   error ("%Hinvalid operands to binary %s (have %qT and %qT)",
Manuel>            location (binary_op), opname, type0, type1);
Manuel> is both more prone to mistakes (people will forget to use %H) and also
Manuel> is less clear (and uglier).

I agree.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]