From: Junio C Hamano Date: Fri, 23 Jun 2006 23:28:47 GMT Subject: Re: [PATCH 0/5] Rework diff options Message-ID: <7vodwj8n9s.fsf@assigned-by-dhcp.cox.net> In-Reply-To: Johannes Schindelin writes: > Hi, > > On Sat, 24 Jun 2006, Timo Hirvonen wrote: > >> This patch series cleans up diff output format options. > > Very good. > > Although I understand that to convert all users to the new convention, it > is sensible to rename the constants, I think it is not good to change > something as DIFF_FORMAT_RAW to OUTPUT_FMT_RAW in the resulting patch. I personally feel that the benefit of being able to make sure you covered everything outweighs the size of initial diff. Thanks Timo. Will take a look.