Issue #5246 - Adding DeflaterPool to GzipHandler.dump

[joakime]

Output looks like ...

GzipHandler@2f4205be{STARTED,min=2048,inflate=-1} - STARTED
...(snip)...
+= DeflaterPool@417ad4f3{STARTED,size=0,capacity=UNLIMITED} - STARTED

Signed-off-by: Joakim Erdfelt joakim.erdfelt@gmail.com

[lachlan-roberts]

LGTM

I think its neater to use String.format

return String.format("%s@%x{%s,size=%d,capacity=%s}",
    getClass().getSimpleName(),
    hashCode(),
    getState(),
    (_pool == null) ? -1 : _pool.size(),
    (_capacity <= 0) ? "UNLIMITED" : _capacity);

this would make it more consistent with the changes for 10, but this works as well.

It might be good to bring back some of the other changes of #5248. Like using the InflaterPool in the GzipHandler as well. But I can do this in a separate PR.

[gregw]

@joakime can this be merged before it goes stale?