Geoff Howard wrote:

[EMAIL PROTECTED] wrote:

unico 2004/02/27 09:27:04

Modified: src/blocks/eventcache/java/org/apache/cocoon/caching/impl
StoreEventRegistryImpl.java
DefaultEventRegistryImpl.java
Log:
fix NPE on first use



FYI I recently re-started making the StoreEventReg default, renaming DefaultEventReg, refactoring inheritance a little (pulled out an AbstractEventRegistry which both inherit from) etc. It's quite unfinished and I have close to no time right now for "play" but just wanted to give you a heads up that I have some refactor work going on. Next chance I get, I'll try to get it in a working state and either commit it or pass it on as a patch for someone else to pick up. I'd like to close the open bug about the serialized data location not being configurable but couldn't stand to hack in configuration when we've already agreed that some refactoring was needed.

Great, the bugzilla entry was also in the back of my mind. I've been using the current default registry in production sites until now but still had on my todo list to use the persistent store delegating one.



Don't quite get what was wrong before this commit by the way. Was this throwing an NPE all this time? (don't think so...)


Yes, the key map never got initialized.

Unico

Reply via email to