[cfe-commits] [PATCH] Improve the warning for cv-qualifiers on free functions.

Ahmed Charles ahmedcharles at gmail.com
Mon Oct 17 09:17:19 PDT 2011


Thanks. Another thing, I'm trying to convince myself that nothing can
come between const and volatile if both are there, is that true or is
it possible to have a token between them? I.e. Should I have two
fixits rather than one?

On Mon, Oct 17, 2011 at 9:05 AM, Douglas Gregor <dgregor at apple.com> wrote:
>
> On Oct 17, 2011, at 1:13 AM, Ahmed Charles wrote:
>
>> Oh, is there a standard way of testing fixits?
>
> Yes, check out test/Fixit. We essentially use -fixit mode to clean up the code, then make sure that the resulting code compiles without warnings/errors.
>
>        - Doug
>
>> On Sun, Oct 16, 2011 at 11:50 PM, Ahmed Charles <ahmedcharles at gmail.com> wrote:
>>> This fixes: http://llvm.org/bugs/show_bug.cgi?id=7824
>>>
>>> I'm not entirely sure that this is the right approach, but after
>>> investigating a while lot, this seems to be the way that Declarators
>>> work. It also seems that the location information isn't preserved past
>>> it's use in Sema, through the Declarator.
>>>
>>
>>
>>
>> --
>> Ahmed Charles
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>



-- 
Ahmed Charles




More information about the cfe-commits mailing list