[PATCH] replace some const std::string & with llvm::StringRef or std::string

[Yaron Keren]

Hi Eugene,

These patches LGTM. Should I commit them for you?

Yaron


2015-06-26 23:33 GMT+03:00 Eugene Kosov <claprix at yandex.ru>:

> Hi.
>
> Thanks for your feedback. I fixed the regression. Here is the new patch.
>
> 26.06.2015, 00:51, "Yaron Keren" <yaron.keren at gmail.com>:
> > Hi Eugene,
> >
> > This patch does not pass clang extra tools regression tests locally,
> >
> > FAIL: Clang Tools :: modularize/NoProblems.modularize (7141 of 19860)
> > Testing: 0 .. 10.. 20.. 30
> > FAIL: Clang Tools :: modularize/NoProblemsDependencies.modularize (7144
> of 19860)
> > Testing: 0 .. 10.. 20.. 30
> > FAIL: Clang Tools :: modularize/NoProblemsCoverage.modularize (7145 of
> 19860)
> > (...)
> >
> > Please check it out.
> >
> > Yaron
> >
> > 2015-06-22 15:25 GMT+03:00 Eugene Kosov <claprix at yandex.ru>:
> >> Hi.
> >>
> >> Here are two patches for clang-tools-extra and for clang which
> simplifies code a little bit and also removes possibility of implicit
> std::string construction.
> >>
> >> --
> >> Eugene
> >>
> >> _______________________________________________
> >> cfe-commits mailing list
> >> cfe-commits at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
> --
> Eugene
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150702/3b9764cf/attachment.html>