This is an archive of the discontinued LLVM Phabricator instance.

clang-format: [js] Support classes.
ClosedPublic

Authored by mprobst on Feb 18 2015, 8:23 AM.

Details

Reviewers
djasper
Summary

This adds support for JavaScript class definitions (again following TypeScript & AtScript style). This only required support for visibility modifiers in JS, everything else was already working.

Diff Detail

Event Timeline

mprobst updated this revision to Diff 20183.Feb 18 2015, 8:23 AM
mprobst retitled this revision from to clang-format: [js] Support classes..
mprobst updated this object.
mprobst edited the test plan for this revision. (Show Details)
mprobst added a reviewer: djasper.
mprobst added subscribers: Unknown Object (MLST), klimek.
djasper added inline comments.Feb 18 2015, 8:29 AM
lib/Format/UnwrappedLineParser.cpp
829–831

As JavaScript also doesn't have function-like macros without trailing semicolon, could you move this to the outer if?

mprobst updated this revision to Diff 20188.Feb 18 2015, 8:35 AM
djasper accepted this revision.Feb 18 2015, 9:16 AM
djasper edited edge metadata.
This revision is now accepted and ready to land.Feb 18 2015, 9:16 AM
djasper closed this revision.Feb 18 2015, 9:16 AM

Submitted as r229701.