This is an archive of the discontinued LLVM Phabricator instance.

Differential D8190

clang-format: Recognize .ts files as JavaScript (TypeScript is a dialect of JavaScript).
ClosedPublic

Authored by mprobst on Mar 9 2015, 7:17 PM.

Details

Reviewers
djasper

Diff Detail

Event Timeline

mprobst updated this revision to Diff 21542.Mar 9 2015, 7:17 PM
mprobst retitled this revision from to clang-format: Recognize .ts files as JavaScript (TypeScript is a dialect of JavaScript)..
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.
mprobst updated this revision to Diff 21645.Mar 10 2015, 3:07 PM

Updated to also change clang-format-diff.py.

djasper accepted this revision.Mar 11 2015, 8:00 AM
djasper edited edge metadata.
This revision is now accepted and ready to land.Mar 11 2015, 8:00 AM
djasper closed this revision.Mar 11 2015, 8:01 AM

Submitted as r231926.

Revision Contents

PathSize
lib/
Format/
3 lines
tools/
clang-format/
2 lines

Diff 21645

lib/Format/Format.cpp

Context not available.
static FormatStyle::LanguageKind getLanguageByFileName(StringRef FileName) { static FormatStyle::LanguageKind getLanguageByFileName(StringRef FileName) {
if (FileName.endswith(".java")) { if (FileName.endswith(".java")) {
return FormatStyle::LK_Java; return FormatStyle::LK_Java;
} else if (FileName.endswith_lower(".js")) { } else if (FileName.endswith_lower(".js") || FileName.endswith_lower(".ts")) {
// JavaScript or TypeScript.
return FormatStyle::LK_JavaScript; return FormatStyle::LK_JavaScript;
} else if (FileName.endswith_lower(".proto") || } else if (FileName.endswith_lower(".proto") ||
FileName.endswith_lower(".protodevel")) { FileName.endswith_lower(".protodevel")) {
Context not available.

tools/clang-format/clang-format-diff.py

Context not available.
help='custom pattern selecting file paths to reformat ' help='custom pattern selecting file paths to reformat '
'(case sensitive, overrides -iregex)') '(case sensitive, overrides -iregex)')
parser.add_argument('-iregex', metavar='PATTERN', default= parser.add_argument('-iregex', metavar='PATTERN', default=
r'.*\.(cpp|cc|c\+\+|cxx|c|cl|h|hpp|m|mm|inc|js|proto' r'.*\.(cpp|cc|c\+\+|cxx|c|cl|h|hpp|m|mm|inc|js|ts|proto'
r'|protodevel|java)', r'|protodevel|java)',
help='custom pattern selecting file paths to reformat ' help='custom pattern selecting file paths to reformat '
'(case insensitive, overridden by -regex)') '(case insensitive, overridden by -regex)')
Context not available.