[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