From: Steven Drake Date: Fri, 26 Feb 2010 00:06:13 GMT Subject: Re: [PATCH] Add `[decorate]' configuration section. Message-ID: In-Reply-To: <7vk4u0rju2.fsf@alter.siamese.dyndns.org> On Thu, 25 Feb 2010, Junio C Hamano wrote: > Steven Drake writes: > > > On Tue, 16 Feb 2010, Junio C Hamano wrote: > > > I was not worried about what your change does. I am worried about > > > protecting what the code after your change currently does from future > > > changes done by other people while you are not actively watching the > > > patches in flight on this list. > > > > But that can easly be with the patch below. > > What does that buy us? "future changes" can revert your patch to > builtin-log.c easily. > > I was talking about having tests in the test suite; any "future changes" > that breaks the output your series has established would not pass if you > specify what the expected output should be there. Of course they can > change the test pattern in their patches, but then we will immediately > know they are changing the output. Sorry I didn't get what you meant! I'll try writing a test. -- Steven