clang-format: [JS] new and delete are valid function names.
Before: someObject.new (); someObject.delete (); After: someObject.new(); someObject.delete(); llvm-svn: 222890
This commit is contained in:
parent
e402d9ef4c
commit
7912123893
|
@ -1671,12 +1671,11 @@ bool TokenAnnotator::spaceRequiredBetween(const AnnotatedLine &Line,
|
|||
if (Right.is(tok::l_paren)) {
|
||||
if (Left.is(tok::r_paren) && Left.is(TT_AttributeParen))
|
||||
return true;
|
||||
return Line.Type == LT_ObjCDecl ||
|
||||
Left.isOneOf(tok::kw_new, tok::kw_delete, tok::semi) ||
|
||||
return Line.Type == LT_ObjCDecl || Left.is(tok::semi) ||
|
||||
(Style.SpaceBeforeParens != FormatStyle::SBPO_Never &&
|
||||
(Left.isOneOf(tok::kw_if, tok::kw_for, tok::kw_while,
|
||||
tok::kw_switch, tok::kw_case) ||
|
||||
(Left.is(tok::kw_catch) &&
|
||||
(Left.isOneOf(tok::kw_catch, tok::kw_new, tok::kw_delete) &&
|
||||
(!Left.Previous || Left.Previous->isNot(tok::period))) ||
|
||||
Left.IsForEachMacro)) ||
|
||||
(Style.SpaceBeforeParens == FormatStyle::SBPO_Always &&
|
||||
|
|
|
@ -361,6 +361,8 @@ TEST_F(FormatTestJS, TryCatch) {
|
|||
|
||||
// But, of course, "catch" is a perfectly fine function name in JavaScript.
|
||||
verifyFormat("someObject.catch();");
|
||||
verifyFormat("someObject.new();");
|
||||
verifyFormat("someObject.delete();");
|
||||
}
|
||||
|
||||
TEST_F(FormatTestJS, StringLiteralConcatenation) {
|
||||
|
|
Loading…
Reference in New Issue