[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