[Diffusion] rL248367: clang-format: Add initial #include sorting capabilities.
Daniel Jasper via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 23 06:13:22 PDT 2015
Thank you. I think I have fixed the issue in r248375.
On Wed, Sep 23, 2015 at 12:25 PM, NAKAMURA Takumi <geek4civic at gmail.com>
> chapuni added subscribers: cfe-commits, chapuni.
> /cfe/trunk/lib/Tooling/Core/Replacement.cpp:408 FirstI may be in the end()
> when SecondI != end(). Then FirstI->getOffset() is invalid.
> I suggest to split the loop into
> - FirstI != First.end() && SecondI != Second.end();
> - Tail-merge (either is end())
> djasper (Author)
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits