[all-commits] [llvm/llvm-project] d45df0: clang-format: [JS] merge import lines.

[Martin Probst via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d45df0d29f7005d3c25357f3982002eaf339f875
      https://github.com/llvm/llvm-project/commit/d45df0d29f7005d3c25357f3982002eaf339f875
  Author: Martin Probst <martin at probst.io>
  Date:   2021-04-14 (Wed, 14 Apr 2021)

  Changed paths:
    M clang/lib/Format/SortJavaScriptImports.cpp
    M clang/unittests/Format/SortImportsTestJS.cpp

  Log Message:
  -----------
  clang-format: [JS] merge import lines.

Multiple lines importing from the same URL can be merged:

    import {X} from 'a';
    import {Y} from 'a';

Merge to:

    import {X, Y} from 'a';

This change implements this merge operation. It takes care not to merge in
various corner case situations (default imports, star imports).

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


  Commit: 4d195f1b4dd6e3978776d69f49840439933a2543
      https://github.com/llvm/llvm-project/commit/4d195f1b4dd6e3978776d69f49840439933a2543
  Author: Martin Probst <martin at probst.io>
  Date:   2021-04-14 (Wed, 14 Apr 2021)

  Changed paths:
    M clang/lib/Format/SortJavaScriptImports.cpp
    M clang/unittests/Format/SortImportsTestJS.cpp

  Log Message:
  -----------
  review comments

track symbol merge status in references to avoid excesive rewrites


Compare: https://github.com/llvm/llvm-project/compare/f29dcbdde10c...4d195f1b4dd6