[cfe-commits] [PATCH] Fix for bug 11917 (crash in SemaInit.cpp)

Aaron Ballman aaron at aaronballman.com
Thu Feb 9 05:37:10 PST 2012


On Thu, Feb 9, 2012 at 6:35 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Thu, Feb 9, 2012 at 4:42 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
>> I happened to notice that Remi Gacogne (rgacogne-free at valombre.net)
>> submitted this bug report with a patch in it.  I've added a test case
>> and created the patch file, and am just looking for a review before I
>> commit.  The patch looks solid to me, and it does rectify the crash.
>> But a second pair of eye never hurts.
>
> The patch adds DOS-style newlines to test/Sema/init.c that contained
> only UNIX-style newlines.  Also, I think you could use
> dyn_cast_or_null.

Good catch, I'll fix that up when I get the chance.

~Aaron




More information about the cfe-commits mailing list