On Nov 10, 2007 11:01 AM, Mike Heath <[EMAIL PROTECTED]> wrote: > Trustin Lee wrote: > > Thanks Mike! > > > > I was actually expecting users to call addMessageEncoder/Decoder multiple > > times: > > > > codec.addMessageEncoder(String.class, SuperMessageEncoder.class); > > codec.addMessageEncoder(Date.class, SuperMessageEncoder.class); > > After putting the change in there, and using it a bit, I'm wondering if > it wouldn't be better to change the API and specify the encoder and > first and then the types using the varargs notation. Something like: > > addMessageEncoder(Class<? extends MessageEncoder> encoder, Class<?>... > messageTypes) > > That way, could add one or many classes in a single call and it's not as > tedious as creating and passing in an Iterable.
I thought the same. Please go ahead! :) Cheers, Trustin -- what we call human nature is actually human habit -- http://gleamynode.net/ -- PGP Key ID: 0x0255ECA6
