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] Fix up libstdc++ build breakage with ldbl-extra.ver


Il 28/02/2012 10:19, Jakub Jelinek ha scritto:
>>> > > -	$(EGREP) -v '#(#| |$$)' $@.tmp | \
>>> > > +	$(EGREP) -v '^[ 	]*#(#| |$$)' $@.tmp | \
>> > 
>> > I don't know this part very well, so I wonder why you have to remove
>> > comments at all...  hence I wonder if sed 's/##.*//;s/# .*//;s/#$//'
>> > (alternation is not portable in sed) would be closer to the original
>> > intentions.
> The reason for comment removal is that we pipe this into the preprocessor,
> which without the removal spits hundreds of
> <stdin>:25:7: error: invalid preprocessing directive #Names
> <stdin>:33:7: error: invalid preprocessing directive #std
> etc. errors.  Preprocessing directives are only recognized at the
> start of the lines, after optional whitespace, so other comments are
> just fine.

Ok then!

Paolo


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