[PATCH] clang-format: Support @autoreleasepool.

Nico Weber thakis at chromium.org
Wed Jun 10 21:58:04 PDT 2015


Can you add a test?

On Wed, Jun 10, 2015 at 4:08 PM, strager <strager.nds at gmail.com> wrote:

> Hi djasper,
>
> Format @autoreleasepool properly for the Attach brace style
> by recognizing @autoreleasepool as a block introducer.
>
> http://reviews.llvm.org/D10372
>
> Files:
>   lib/Format/UnwrappedLineParser.cpp
>   unittests/Format/FormatTest.cpp
>
> Index: lib/Format/UnwrappedLineParser.cpp
> ===================================================================
> --- lib/Format/UnwrappedLineParser.cpp
> +++ lib/Format/UnwrappedLineParser.cpp
> @@ -657,6 +657,16 @@
>        nextToken();
>        addUnwrappedLine();
>        return;
> +    case tok::objc_autoreleasepool:
> +      nextToken();
> +      if (FormatTok->Tok.is(tok::l_brace)) {
> +        if (Style.BreakBeforeBraces == FormatStyle::BS_Allman ||
> +            Style.BreakBeforeBraces == FormatStyle::BS_GNU)
> +          addUnwrappedLine();
> +        parseBlock(/*MustBeDeclaration=*/false);
> +      }
> +      addUnwrappedLine();
> +      return;
>      case tok::objc_synchronized:
>        nextToken();
>        if (FormatTok->Tok.is(tok::l_paren))
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150610/3f013fce/attachment.html>


More information about the cfe-commits mailing list