Currently the base class for all codecs (Codec) is not public which makes it
impossible to write own codecs.
I think this was kept private originally out of concern that the API might not be stable. But CodecFactory is public, so it doesn't make much sense for Codec to be private. I'm +1 for making this change. Would you like to provide a patch? Should we add a test that implements a Codec in a different package, perhaps one that just performs a bitwise-NOT of the data?
Performed modifications as specified in the previous comment. Had to modify the three implementations of Codec in order to allow extend and override of new method signatures. Added JUnit test and Custom codec implementation in test subpackage.
Hope everything is fine, please review patch.
We ought to remove the javadoc comment in Codec.java that states it's an internal API. Other than that this looks good to me.
Unless someone objects, I'll commit this soon.
Same as before but without the javadoc comment in Codec.java that stated it's an internal API
I committed this. Thanks, Rui!