>>
>>
> So our test suite lacks this test?
>
>> Anyhow, in r26813 I added an intialization and also tweaked
>> drive_merge_report_editor() to check for a possible NULL
>> children_with_mergeinfo (not really necessary right now, but you check
>> it elsewhere in that function, so in the name of safety and
>> consistency...
>>
>>
> Just wondering why you didn't add that test in r26813? If you don't have
> time, why not add an XFail-ing place holder explaining in comments the
> above scenario, then we can still add it later.
>
> Lieven
>
>
Lieven,
Yes I *love* tests that shows detects all the problems in my code well
in advance(This is what exactly I feel when I change something in
merge-tracking code). But writing a testcase for short lived bugs of
this nature would be an overkill.
With regards
Kamesh Jayachandran
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 28 09:43:31 2007