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] macro redefinition with different token count


Loren James Rittle <rittle@latour.rsch.comm.mot.com> writes:

>>	* cppmacro.c (warn_of_redefinition): Handle cases where the two
>>	definitions have different numbers of tokens.
>
> Thanks DJ, I can confirm that this fixes the test case in
> preprocessor/11022 (thus consider adding an annotation and closing
> that PR when you commit).  I revised the posted test case into dejagnu
> form (consider committing it along with your patch since this problem
> wasn't covered by the existing tests) and covered the related oddity.
>
> (I'd say that your fix is obviously correct, but I'm only adding some bits.)

It's correct.  Please do check both the patch and the testcase in, and
close the PR.

zw


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