[dev] [cvs] commit: imp/lib/IMAP MailboxCache.php

[Michael M Slusarz]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>>   Log:
>>   Only (un)serialize array if we need to.
>
> Is there any advantage to this? The old code would work regardless,
> and introducing if checks based on Horde versions is fragile and just
> hard to maintain. I'd _strongly_ prefer to avoid it.

Of course.  Horde_Cache_file:: is completely unusable otherwise for  
development purposes.  i.e. you can't switch backends transparently  
(for example, zps4 driver accepts an array but file driver doesn't).

michael

___________________________________
Michael Slusarz [slusarz at horde.org]