[dev] [cvs] commit: framework/Cache/Cache file.php

[Chuck Hagenbuch]

Quoting Michael M Slusarz <slusarz at horde.org>:

> There is absolutely no harm in double serializing data.  Unserializing
> data we *know* has been serialized (see set()) is perfectly fine.  Not
> sure how this breaks caching of string data.

Sure there is. Telling PHP to unserialize a large string - say an XML  
document - is going to be inefficient and use a lot of memory. We  
store things like generated iCalendar data in the cache - we don't  
gain anything by serializing them.

> Again, there is a function_exists() check.

It's missing in set().

-chuck

-- 
"we are plastered to the windshield of the bus that is time." - Chris