r199198 - clang-format: Fix bug introduced in r198871.

Jordan Rose jordan_rose at apple.com
Tue Jan 14 09:04:12 PST 2014


On Jan 14, 2014, at 1:53 , Daniel Jasper <djasper at google.com> wrote:

> Author: djasper
> Date: Tue Jan 14 03:53:07 2014
> New Revision: 199198
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=199198&view=rev
> Log:
> clang-format: Fix bug introduced in r198871.
> 
> We cannot simply change the start column to accomodate for the @ in an
> ObjC string literal as that will make clang-format happily violate the
> column limit.
> 
> Use a different workaround instead. However, a better long-term
> solution might be to join the @ and the rest of the literal into a
> single token.

Eh, you're going to have this problem with every other @ literal now: @123, @YES (expands to @__objc_yes), @(cos(x)). It pretty much always wants to stick to its following token, whatever that may be.

Jordan



More information about the cfe-commits mailing list