[Box Backup-dev] COMMIT r840 - box/chris/merge/bin/bbackupd
Ben Summers
boxbackup-dev@fluffy.co.uk
Thu, 31 Aug 2006 14:04:27 +0100
On 31 Aug 2006, at 13:52, Martin Ebourne wrote:
> Ben Summers <ben@fluffy.co.uk> wrote:
>> You've already re-written the main if statement for whether to
>> update a
>> file or not.
>
> Tell me about it! I confess to being horrified when that one came
> through. I'd have rejected it had I not agreed with Chris that the
> original was more difficult to understand and impossible to debug.
>
> But it took me ages going over it bit by bit, and twice as well, to
> try and convince myself it was correct.
I think it's correct too. And the tests still pass. The original was
not one of my finest hours, I think it ended up that way because I
was trying to work out the best way to do it as I went along.
Ben