This is an archive of the discontinued LLVM Phabricator instance.

clang-format: [JS] Sort imports case insensitive.
ClosedPublic

Authored by mprobst on Jul 8 2016, 11:37 AM.

Details

Summary

ASCII case sorting does not help finding imported symbols quickly, and it is common to have e.g. class Foo and function fooFactory exported/imported from the same file.

Diff Detail

Event Timeline

mprobst updated this revision to Diff 63268.Jul 8 2016, 11:37 AM
mprobst retitled this revision from to clang-format: [JS] Sort imports case insensitive..
mprobst updated this object.
mprobst added a reviewer: djasper.
mprobst added a subscriber: cfe-commits.
mprobst updated this revision to Diff 63271.Jul 8 2016, 11:43 AM
  • clang-format: [JS] support trailing commas in imports.
djasper accepted this revision.Jul 8 2016, 11:10 PM
djasper edited edge metadata.

Looks good.

This revision is now accepted and ready to land.Jul 8 2016, 11:10 PM
This revision was automatically updated to reflect the committed changes.