OK. I think that exposing the threshold=off functionality as "pause" is more of a gui terminology thing. So, I am voting with my code to go with threshold. I will shortly check in a "Thresholdable" interface (ick, but it is better than IThreshold), similar to the one you suggest Paul. I have changed set/getLevel to set/getThreshold. Yes, we could leave it as set/getLevel and have Logger, AppenderSkeleton, and Reciever all implement the interface, but I don't know if we want to make that big of a change to the logger interface. AppenderSkeleton already has the methods (except that is uses Priority instead of Level).
I will put it into the sandbox, and we can move it in after folks have had a chance to digest/change it. At that time, we should modify AppenderSkeleton to implement it. thanks! -Mark ----- Original Message ----- From: "Paul Smith" <[EMAIL PROTECTED]> To: "'Log4J Developers List'" <[EMAIL PROTECTED]> Sent: Sunday, June 22, 2003 10:17 PM Subject: RE: Latest Receiver/Plugin commits to sandbox > > Why does using a threshold=off not achieve the same as "pause"? > > It does, but for semantics it might be useful to have this 'behaviour', even > if it's implemented by having a Threshold set to off. I also agree it's > very blurred. > > Could we convert this Pauseable interface then into a Thresholdable? (with > setLevel/getLevel). From a GUI perspective I can implement pause as setting > Threshold=Off. > > If we use this interface, and get Appender/Logger et all all supporting this > interface, then there is nothing to stop us having all the lugin, Category, > and Appender definitions sitting in a nice GUI tree for ease of management. > > Anyway, to summarize, +1 on Pause gets converted to Threshold.OFF. > > Paul > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
