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. WDYT? -Mike
