[cfe-commits] [patch] Initialize all of struct tm in some libc++ locale functions

Jeffrey Yasskin jyasskin at gmail.com
Sun Feb 19 09:14:55 PST 2012


On Sun, Feb 19, 2012 at 3:20 AM, Chandler Carruth <chandlerc at google.com> wrote:
> Why '{0}' instead of '{}' which is a bit more idiomatic in C++?

No big reason. There were a couple other places that used {0}, and I
personally like the extra reminder that it's 0 the rest of the fields
will be initialized with.

> Otherwise looks good...
>
> On Feb 19, 2012 1:59 AM, "Jeffrey Yasskin" <jyasskin at gmail.com> wrote:
>>
>> One of the uninitialized fields was causing a segfault.
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>



More information about the cfe-commits mailing list