[cfe-dev] Adding access to array size expressions in ConstantArrayType.

Eli Friedman eli.friedman at gmail.com
Sat Jul 4 03:31:47 PDT 2009


On Sat, Jul 4, 2009 at 12:31 AM, Enea Zaffanella<zaffanella at cs.unipr.it> wrote:
> Enea Zaffanella wrote:
>>
>> Eli Friedman wrote:
>>>
>>> On Tue, Jun 30, 2009 at 12:35 PM, Enea Zaffanella<zaffanella at cs.unipr.it>
>>> wrote:
>>
>> [...]
>>
>>> Unlike the other array types you're adding Brackets to,
>>> IncompleteArrayType gets canonicalized; we don't want a source range
>>> on a canonical IncompleteArrayType, and we don't want to end up with
>>> the wrong SourceRange on a non-canonical IncompleteArrayType.  It's a
>>> little tricky, so if the issues I'm talking about aren't obvious, I'd
>>> suggest skipping adding the bracket locations to IncompleteArrayType
>>> for now.
>>>
>>> Otherwise, this is looking good!
>>>
>>> -Eli
>>
>> OK, here is the patch revised along your suggestions.
>>
>> Enea.
>
> I got no feedback on my last revision of the patch and it seems that the
> patch was not applied. Is there something wrong with it? Or should I just
> wait a bit more?

Nothing wrong with it; I'll take a final look at it and apply it
sometime in the next couple days.

-Eli




More information about the cfe-dev mailing list