[cfe-dev] Cleanup/Bugfixing in SemaInit.cpp

[Eli Friedman]

On Mon, May 19, 2008 at 10:36 AM, Steve Naroff <snaroff at apple.com> wrote:
> No worries. Since you/I worked on CheckInitializerListTypes, it's really
> good to have you review/enhance CheckInitList. I find this part of the C
> language very tricky. fyi...I'm in the process of moving my family and
> Apple's World Wide Developers conference is coming up soon (as a result,
> CheckInitList hasn't received my attention for a couple weeks now).

Okay.

> I noticed this patch doesn't enable any of these changes yet (i.e. you
> didn't change Sema::CheckInitializerTypes()). Was this intentional?

The enabling patch is trivial; I'll do it in a separate commit.

> I haven't reviewed the patch closely, however I think it moves the ball
> forward. Please commit (and we can iterate if necessary...).

Okay, cool.

> Thanks again - I greatly appreciate the work you are doing!

Thanks.

-Eli