Below is the patch for issue 485.As part of this patch I also cleaned up some of the warnings given by Eclipse, made some minor code improvements, and started the Logger refactor to remove the wrapped methods as per a TODO note.On that note, it would probably be good to look at moving the Logger invocation to be on a final static private class by class basis. This seems to be the standard way to go.

Of particular note, I removed the ChannelPipelineCoverage("one") annotation which was deprecated. I was able to do this since the new Sharable annotation should only be used when one wants to share an object. However the ChannelPipelineCoverage("one") annotation means you must create a new instance of the annotated handler type for each new channel. Thus my interpretation was that the code could simply be removed.