[llvm] r304083 - Added braces to address gcc warning: suggest explicit braces to avoid ambiguous 'else' [-Wdangling-else]. NFC.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Sat May 27 22:30:54 PDT 2017


EXPECT_STREQ should already be using this as far as I can tell.

~Craig

On Sat, May 27, 2017 at 10:17 PM, Davide Italiano via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> On Sat, May 27, 2017 at 10:07 PM, Galina Kistanova <gkistanova at gmail.com>
> wrote:
> > I did not want to fix googletest at this point. But if I'll end up doing
> so,
> > I'll fix this one too.
> >
> >
>
> I'd rather not duplicate the fix for each callsite and make thisan O(N)
> problem.
> I guess you can just replicate Craig's solution for EXPECT_STREQ instead.
>
> Thanks,
>
> --
> Davide
>
> "There are no solved problems; there are only problems that are more
> or less solved" -- Henri Poincare
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170527/161b2798/attachment.html>


More information about the llvm-commits mailing list