[PATCH] D73335: clang-format: [JS] options for arrow functions.

Tue Jan 28 06:05:12 PST 2020

mprobst marked 2 inline comments as done.
mprobst added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:3165
+      }
+    }
+
----------------
RKSimon wrote:
> @mprobst - this is breaking buildbots: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/27196/steps/bootstrap%20clang/logs/stdio
> 
> Please can you remove the default case and add a suitable llvm_unreachable() after the switch statement ?
Sorry for the breakage, and thanks to Jinsong Ji for fixing while I was OOO.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73335/new/

https://reviews.llvm.org/D73335