On Wed, Jul 16, 2014 at 8:07 PM, Alp Toker <a...@nuanti.com> wrote: > My reply here was me taking my earliest opportunity to engage and >> discuss. =) >> > > I don't understand your sudden expectation to be copied into the > discussion here. It's a module you haven't committed to at all recently or > as far back as I can remember. I've been relatively active in this corner > of the codebase and the change was discussed and approved on the commits > list. >
There were two changes here. One (the file reorganization) was not discussed and not approved. For the other, only two hours passed between the first suggestion of the change and the commit, the discussion happened on an unrelated patch review thread, and it removed a supported configuration option. The level and duration of preceding discussion was not sufficient for either of these changes.
_______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
