<div dir="ltr">Looks good to me, please land.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 15, 2014 at 1:45 AM, Anders Rönnholm <span dir="ltr"><<a href="mailto:Anders.Ronnholm@evidente.se" target="_blank">Anders.Ronnholm@evidente.se</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping. Is it ok to commit this one?<br>
<br>
//Anders<br>
________________________________________<br>
Från: Anders Rönnholm<br>
Skickat: den 30 april 2014 08:16<br>
Till: Nico Weber<br>
Cc: Jordan Rose; <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
Ämne: SV: [PATCH] String plus char<br>
<div class="HOEnZb"><div class="h5"><br>
I have run it on LibreOffice and LLVM with no bugs found and no false positives either.<br>
<br>
//Anders<br>
________________________________________<br>
Från: <a href="mailto:thakis@google.com">thakis@google.com</a> [<a href="mailto:thakis@google.com">thakis@google.com</a>] f&#246;r Nico Weber [<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>]<br>
Skickat: den 25 april 2014 17:36<br>
Till: Anders Rönnholm<br>
Cc: Jordan Rose; <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
Ämne: Re: [PATCH] String plus char<br>
<br>
Have you run this warning on some large codebase (test codebases we've<br>
used in the past include LLVM itself, Chromium, Firefox, OpenOffice)<br>
and measured how many bugs and false positives it causes?<br>
<br>
On Fri, Apr 25, 2014 at 3:24 AM, Anders Rönnholm<br>
<<a href="mailto:Anders.Ronnholm@evidente.se">Anders.Ronnholm@evidente.se</a>> wrote:<br>
> Your right. I think it's a left over from when i first tried to cast it to an arraysubscriptexpr.<br>
><br>
> Here is a new patch with array type removed.<br>
><br>
> //Anders<br>
> ________________________________________<br>
> Från: Jordan Rose [<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>]<br>
> Skickat: den 24 april 2014 18:43<br>
> Till: Anders Rönnholm<br>
> Cc: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> Ämne: Re: [PATCH] String plus char<br>
><br>
> If we're not looking through implicit casts, when would there ever be a case when the StringRefExpr is a constant array type? Shouldn't it always decay to a pointer?<br>
><br>
> Jordan<br>
><br>
><br>
> On Apr 17, 2014, at 5:19 , Anders Rönnholm <<a href="mailto:Anders.Ronnholm@evidente.se">Anders.Ronnholm@evidente.se</a>> wrote:<br>
><br>
>> Hi,<br>
>><br>
>> I have made some improvements on the string plus char warning.<br>
>><br>
>> It can now also handle arrays, struct members and return values from function calls.<br>
>><br>
>> e.q<br>
>><br>
>> ab->a + 'b';<br>
>><br>
>> foo() + 'c';<br>
>><br>
>> char *strArr2[] = {"ac","dc"};<br>
>> str = strArr2[0] + 'c';<br>
>><br>
>> //Anders<br>
>> <stringpluschar.diff>_______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div></div>