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

On Jan 7, 2013, at 10:52 , Daniel Jasper <[email protected]> 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 <[email protected]> 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 <[email protected]> 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
> 
> 

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to