Re: [Duplicity-talk] unused assert

[Kenneth Loafman]

Jason Dusek wrote:
> Kenneth Loafman <address@hidden> wrote:
>> At some point get_sorted_chains() must have screwed up.
>> Perhaps it would be wise to assume it might happen again.
> 
>   Let me try again. The original code is as follows:
> 
>     self.all_backup_chains = backup_chains
> 
>     assert len(backup_chains) == len(self.all_backup_chains)
> 
>   Clearly, this is true no matter what `get_sorted_chains` did.
>   So the assert is not really testing anything.

Yes, sorry, you are correct the assert is not doing anything useful at
all.  Perhaps it did at some time, but not now.

...Ken

signature.asc
Description: OpenPGP digital signature