Apologies for the delay, I'm a little behind on things this week..
Bill Moseley wrote:
Unfortunately, Cache::Entry doesn't really specify a return value for set(), but common backends like Memcached and FastMmap do seem to return a value indicating success. Not sure if there are other commonly used backends that would not have a meaningful return value. The plugin doesn't define the return from cache_set, either. As it is now, cache_set can silently fail.
It's useful to know the return value in some cases, so I'm all for a patch which propagates that.
I wonder if it would be safe to warn on ->set() failing in the plugin.
I think for the common case, this is probably good behavior, however I think there should be a config setting which could disable this warning..
Patches for both of the above would be welcome. Cheers t0m _______________________________________________ List: Catalyst@lists.scsys.co.uk Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/catalyst@lists.scsys.co.uk/ Dev site: http://dev.catalyst.perl.org/
