[PATCH] An attempt to make the search algorithm easier to understand.
Daniel Jasper
djasper at google.com
Wed Feb 13 01:48:32 PST 2013
================
Comment at: lib/Format/Format.cpp:644
@@ +643,3 @@
+ }
+ LineState State;
+ bool NewLine;
----------------
Daniel Jasper wrote:
> Manuel Klimek wrote:
> > Daniel Jasper wrote:
> > > Might be nice to reflect in the name that this is the node that this edge leads to. Maybe "NextState"?
> > Interesting view. I actually see this as a node with an *incoming* edge :) (that's why the link is called Previous)
> That few makes sense except then this would be a StateNode and not a StateEdge, right?
s/few/view
http://llvm-reviews.chandlerc.com/D398
BRANCH
format-penalty
ARCANIST PROJECT
clang
More information about the cfe-commits
mailing list