[cfe-commits] r171732 - in /cfe/trunk: include/clang/Format/Format.h lib/Format/Format.cpp lib/Format/UnwrappedLineParser.cpp lib/Format/UnwrappedLineParser.h unittests/Format/FormatTest.cpp

[Jordan Rose]

Okay, glad to hear it's configurable. Thanks, Daniel.


On Jan 7, 2013, at 10:52 , Daniel Jasper <djasper at google.com> wrote:

> clang-format is configured to reduce all consecutive blank lines to a single blank line. This is configurable, but we didn't have a strong reason to set a number greater than 1 yet. IMO, for this code, it does not make a difference.
> 
> 
> On Mon, Jan 7, 2013 at 6:26 PM, Jordan Rose <jordan_rose at apple.com> wrote:
> This removed double blank lines, which sometimes separate different logical sections of code. Was that intentional?
> 
> 
> On Jan 7, 2013, at 5:26 , Daniel Jasper <djasper at google.com> wrote:
> 
> > Author: djasper
> > Date: Mon Jan  7 07:26:07 2013
> > New Revision: 171732
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=171732&view=rev
> > Log:
> > Reformat clang-formats source code.
> >
> > All changes done by clang-format itself. No functional changes.
> >
> > Modified:
> >    cfe/trunk/include/clang/Format/Format.h
> >    cfe/trunk/lib/Format/Format.cpp
> >    cfe/trunk/lib/Format/UnwrappedLineParser.cpp
> >    cfe/trunk/lib/Format/UnwrappedLineParser.h
> >    cfe/trunk/unittests/Format/FormatTest.cpp
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130107/b343337d/attachment.html>