> Along those lines, it might be nice to add an "isActivated" read only > property, which is set by the Plugin on activateOptions call. > What do you > think?
This just shows how long it's been since I've been in this area (or any area) of the log4j code. There already IS an isActive method.... Perhaps the PluginRegistry.addPlugin method could either a) call activateOptions if it's not active yet, OR throw IllegalStateException, as a sanity check to enforce the contract that Plugins must be activated before being added. ? Paul --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
