[all-commits] [llvm/llvm-project] 4f4f27: [clang-format] don't break between function and fu...

Krasimir Georgiev via All-commits all-commits at lists.llvm.org
Tue Aug 3 02:19:38 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4f4f2783056fd01182740251b2ce8a77b12684b3
      https://github.com/llvm/llvm-project/commit/4f4f2783056fd01182740251b2ce8a77b12684b3
  Author: Krasimir Georgiev <krasimir at google.com>
  Date:   2021-08-03 (Tue, 03 Aug 2021)

  Changed paths:
    M clang/lib/Format/ContinuationIndenter.cpp
    M clang/unittests/Format/FormatTestJS.cpp

  Log Message:
  -----------
  [clang-format] don't break between function and function name in JS

The patch https://reviews.llvm.org/D105964 (https://github.com/llvm/llvm-project/commit/58494c856a15f5b0e886c7baf5d505ac6c05dfe5)
updated detection of function declaration names. It had the unfortunate
consequence that it started breaking between `function` and the function
name in some cases in JavaScript code.

This patch addresses this.

Reviewed By: MyDeveloperDay, owenpan

Differential Revision: https://reviews.llvm.org/D107267




More information about the All-commits mailing list