[cfe-commits] [PATCH] Initial version of formatting library

Dmitri Gribenko gribozavr at gmail.com
Thu Oct 25 08:42:57 PDT 2012

  The design document states that the lexer-only approach is very limited.  This patch is based purely on the lexer.  How is this going to be extended to handle ambiguous constructs that require an AST?  Is there an intention to invoke the parser and resynchronize the AST with the token stream somehow?

Comment at: unittests/Format/Makefile:15
@@ +14,3 @@
+USEDLIBS = clangFormat.a clangTooling.a clangFrontend.a clangSerialization.a \
+           clangDriver.a clangParse.a clangRewriteCore.a
+           clangRewriteFrontend.a clangSema.a clangAnalysis.a clangEdit.a \
Missing backslash at the end of the line?

Comment at: lib/Format/Format.cpp:9
@@ +8,3 @@
+//  This is EXPERIMENTAL code under heavy development. It is not in a state yet,
Doxygen and \file this comment please.


More information about the cfe-commits mailing list