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

Chandler Carruth chandlerc at google.com
Sun Dec 2 01:17:18 PST 2012


On Sat, Dec 1, 2012 at 9:14 PM, Sean Silva <silvas at purdue.edu> wrote:

> On Sat, Dec 1, 2012 at 7:20 PM, Daniel Jasper <djasper at google.com> wrote:
> > Yes. I will fix / document it.
>
> I feel vindicated ;)
>
> On Sat, Dec 1, 2012 at 7:20 PM, Daniel Jasper <djasper at google.com> wrote:
> > Basic question about the approach, just
> > to be sure we are on the same page: Since you are commenting on the
> > current version of the github repository, should I polish that a bit
> > and send an updated version out for review?
>
> I was thinking this too. IMO it makes sense to avoid putting "stale"
> code into the tree. Was the github fork just a temporary thing due to
> the blockage on this patch, or do you plan to continue development out
> of tree with code drops? Personally, I would like to see all of the
> current work on github polished and committed, with all new
> development happening incrementally on trunk (to avoid divergence).


We are always trying to develop incrementally on trunk. *Always*. =]
Anything like this is one of two things:

1) The code is too rough to even make sense in code review yet, and really
it's just a personal hacking backlog. See my SROA rewrite.
2) There is a patch in really extended review, and so incremental progress
get's backed up behind it. We try to do it in the open even so just so that
folks know where we're going, etc.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121202/27cbfeb5/attachment.html>


More information about the cfe-commits mailing list