[PATCH] D81255: [clang-format] treat 'lock' as a keyword for C# code
Phabricator via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 8 05:58:34 PDT 2020
This revision was automatically updated to reflect the committed changes.
Closed by commit rGa67949913a6b: [clang-format] treat 'lock' as a keyword for C# code (authored by Jonathan Coe <jbcoe at google.com>).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81255/new/
https://reviews.llvm.org/D81255
Files:
clang/lib/Format/TokenAnnotator.cpp
clang/unittests/Format/FormatTestCSharp.cpp
Index: clang/unittests/Format/FormatTestCSharp.cpp
===================================================================
--- clang/unittests/Format/FormatTestCSharp.cpp
+++ clang/unittests/Format/FormatTestCSharp.cpp
@@ -741,6 +741,9 @@
})",
Style);
+ // space after lock in `lock (processes)`.
+ verifyFormat("lock (process)", Style);
+
Style.SpacesInSquareBrackets = true;
verifyFormat(R"(private float[ , ] Values;)", Style);
verifyFormat(R"(string dirPath = args?[ 0 ];)", Style);
Index: clang/lib/Format/TokenAnnotator.cpp
===================================================================
--- clang/lib/Format/TokenAnnotator.cpp
+++ clang/lib/Format/TokenAnnotator.cpp
@@ -3080,7 +3080,8 @@
// space between keywords and paren e.g. "using ("
if (Right.is(tok::l_paren))
- if (Left.isOneOf(tok::kw_using, Keywords.kw_async, Keywords.kw_when))
+ if (Left.isOneOf(tok::kw_using, Keywords.kw_async, Keywords.kw_when,
+ Keywords.kw_lock))
return Style.SpaceBeforeParens == FormatStyle::SBPO_ControlStatements ||
spaceRequiredBeforeParens(Right);
} else if (Style.Language == FormatStyle::LK_JavaScript) {
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81255.269190.patch
Type: text/x-patch
Size: 1207 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200608/458d436c/attachment.bin>
More information about the cfe-commits
mailing list