sylvestre.ledru added a comment.
@Typz
I think this should be part of the release notes for v8.
This is changing the output on some code base and this is a new feature.
By the way, an example in the doc would be great (including with the
configuration)
merci!
Repository:
rL LLVM
CHANGES S
This revision was automatically updated to reflect the committed changes.
Closed by commit rL343602: clang-format: better handle statement macros
(authored by Typz, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D33440?vs=158308&id=167
Typz updated this revision to Diff 158308.
Typz marked an inline comment as done.
Typz added a comment.
Herald added a subscriber: acoomans.
Regenerate documentation
Repository:
rC Clang
https://reviews.llvm.org/D33440
Files:
docs/ClangFormatStyleOptions.rst
include/clang/Format/Format.h
Typz updated this revision to Diff 147293.
Typz marked an inline comment as done.
Typz added a comment.
Address review comments
Repository:
rC Clang
https://reviews.llvm.org/D33440
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/FormatToken.h
lib/Format/FormatTo
Typz marked an inline comment as done.
Typz added inline comments.
Comment at: lib/Format/Format.cpp:647-648
LLVMStyle.SortUsingDeclarations = true;
+ LLVMStyle.StatementMacros.push_back("Q_UNUSED");
+ LLVMStyle.StatementMacros.push_back("QT_REQUIRE_VERSION");
alexfh added a comment.
There are still outstanding comments.
Repository:
rC Clang
https://reviews.llvm.org/D33440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz updated this revision to Diff 147024.
Typz added a comment.
Herald added a subscriber: mgrang.
Rebase on latest master
Repository:
rC Clang
https://reviews.llvm.org/D33440
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/FormatToken.h
lib/Format/FormatTokenL
alexfh added inline comments.
Comment at: lib/Format/Format.cpp:647-648
LLVMStyle.SortUsingDeclarations = true;
+ LLVMStyle.StatementMacros.push_back("Q_UNUSED");
+ LLVMStyle.StatementMacros.push_back("QT_REQUIRE_VERSION");
What's the reason to have these
Typz updated this revision to Diff 132837.
Typz added a comment.
Use StatementMacro detection to improve brace type detection heuristics (in
UnwrappedLineParser::calculateBraceTypes).
https://reviews.llvm.org/D33440
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/Fo
Typz added a comment.
ping?
https://reviews.llvm.org/D33440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz added a comment.
ping?
https://reviews.llvm.org/D33440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz updated this revision to Diff 18.
Typz marked 5 inline comments as done.
Typz added a comment.
Address review comments
https://reviews.llvm.org/D33440
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/FormatToken.h
lib/Format/FormatTokenLexer.cpp
lib/Forma
Typz added a comment.
In https://reviews.llvm.org/D33440#920205, @djasper wrote:
> Out of curiosity, will this be able to fix the two situations that you get
> for python extension?
> There, you usually have a PyObject_HEAD with out semicolon in a struct and
> than a PyObject_HEAD_INIT(..) in
djasper added a comment.
Out of curiosity, will this be able to fix the two situations that you get for
python extension?
There, you usually have a PyObject_HEAD with out semicolon in a struct and than
a PyObject_HEAD_INIT(..) in a braced init list. More info:
http://starship.python.net/crew/mwh
Typz added a comment.
ping?
https://reviews.llvm.org/D33440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz updated this revision to Diff 120084.
Typz added a comment.
rebase
https://reviews.llvm.org/D33440
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/FormatToken.h
lib/Format/FormatTokenLexer.cpp
lib/Format/FormatTokenLexer.h
lib/Format/UnwrappedLineParser.cp
Typz updated this revision to Diff 115801.
Typz marked 5 inline comments as done.
Typz added a comment.
Add tests.
Replace sorted list with hashtable.
https://reviews.llvm.org/D33440
Files:
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/FormatToken.h
lib/Format/FormatTok
Typz added inline comments.
Comment at: lib/Format/FormatTokenLexer.cpp:642
tok::pp_define) &&
-std::find(ForEachMacros.begin(), ForEachMacros.end(),
- FormatTok->Tok.getIdentifierInfo()) != ForEachMacros.end()) {
- FormatTok->Type = T
djasper added inline comments.
Comment at: lib/Format/FormatTokenLexer.cpp:642
tok::pp_define) &&
-std::find(ForEachMacros.begin(), ForEachMacros.end(),
- FormatTok->Tok.getIdentifierInfo()) != ForEachMacros.end()) {
- FormatTok->Type
krasimir added inline comments.
Comment at: unittests/Format/FormatTest.cpp:2407
+ "int b = 0;",
+ Style);
}
Please add tests where:
- the macro occurs inside a function body, thus having nontrivial indentation
- there are two macros
20 matches
Mail list logo
