On Wed, Aug 31, 2016 at 10:03:03AM -0700, Kevin J. McCarthy wrote: > On Sat, Aug 13, 2016 at 05:30:52PM -0400, Damien Riegel wrote: > > The purpose of this pachset is to rewrite the way Mutt handles logging > > when compiled with the debug mode. > > Just to add a quick comment, I have noted this patch series and will get > to it. > > However, from my point of view, rewriting the debug logging is very low > priority. I don't much like to see massive changes like this without a > good reason for it. (IMHO) It's a large amount of code to review and a > low payoff. > > Additionally, although I don't much like to say things like this, I'm > not too happy with your track record so far. You came to the list > saying you wanted to work on refactoring the mailbox operations api. > > Yet, half-way through you complained to me that you were demotivated > because no one was helping you, and you bailed out. At the very least, > I would have hoped you would have addressed my comments in > <[email protected]> and would have finished up > mx_open_mailbox_append() so we could remove calls to mx_get_ops() > everywhere.
I'll get back to it, but I made these changes in parallel and I wanted to get them out before I forgot about it. Last months were pretty busy but I have time again to work on mutt. As for your other comments, I will take them into account. Sorry for the attitude. Thanks, -- Damien
