Also now public class AbstractLoggerProviderWrapper extends
AbstractLoggerProvider but it is not abstract.

If it is intended to be, let's declare it as such or drop the "Abstract"
from the name.

Gary


On Mon, Mar 24, 2014 at 1:59 AM, Gary Gregory <[email protected]>wrote:

> In the same vein: AbstractLogger -> AbstractLoggerProvider since it
> implements LoggerProvider.
>
> Gary
>
>
> On Mon, Mar 24, 2014 at 12:26 AM, Matt Sicker <[email protected]> wrote:
>
>> Sounds like a good idea.
>>
>>
>> On 23 March 2014 21:20, Gary Gregory <[email protected]> wrote:
>>
>>> Now that AbstractLoggerWrapper looks like this:
>>>
>>> public class AbstractLoggerWrapper extends AbstractLogger {
>>>
>>>     /**
>>>      * The wrapped Logger.
>>>      */
>>>     protected final LoggerProvider logger;
>>>
>>> Should rename AbstractLoggerWrapper to LoggerProviderWrapper?
>>>
>>> Gary
>>>
>>> --
>>> E-Mail: [email protected] | [email protected]
>>> Java Persistence with Hibernate, Second 
>>> Edition<http://www.manning.com/bauer3/>
>>> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
>>> Spring Batch in Action <http://www.manning.com/templier/>
>>> Blog: http://garygregory.wordpress.com
>>> Home: http://garygregory.com/
>>> Tweet! http://twitter.com/GaryGregory
>>>
>>
>>
>>
>> --
>> Matt Sicker <[email protected]>
>>
>
>
>
> --
> E-Mail: [email protected] | [email protected]
> Java Persistence with Hibernate, Second 
> Edition<http://www.manning.com/bauer3/>
> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
> Spring Batch in Action <http://www.manning.com/templier/>
> Blog: http://garygregory.wordpress.com
> Home: http://garygregory.com/
> Tweet! http://twitter.com/GaryGregory
>



-- 
E-Mail: [email protected] | [email protected]
Java Persistence with Hibernate, Second Edition<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Reply via email to