On Mon, May 09, 2016 at 10:53:06AM -0400, Damien Riegel wrote: > On Sun, May 08, 2016 at 12:58:27PM -0500, Derek Martin wrote: > > Finally got around to this. The patch is obviously quite long; I > > suppose there's nothing to be done about that. > > [...] > but it doesn't make sense to change all M_* to MUTT_* in one giant > commit. It makes it impossible to sent it to the list or to review it > thoroughly.
Thanks for your input Damien. In this case, I'm okay with it because the change was automated, and I verified the patch against the script. I've also checked the symbols changed, and done some spot checking. Lastly, Derek has been around in mutt-dev a long time so I trust him. :-) Even split across multiple commits, the end result of the change is still a bit of a mess for external patches, so if we're doing it I'm fine with doing it in one automated swoop. -- Kevin J. McCarthy GPG Fingerprint: 8975 A9B3 3AA3 7910 385C 5308 ADEF 7684 8031 6BDA
signature.asc
Description: PGP signature
