Alexei Fedotov wrote:
I have completed changes [1] based on your ideas of using
ByteArrayInputStream child for exposing its underlying buffer as well
as fixed few of my earlier TODO comments. I found the contract of the
underlying buffer capturing close to ByteBuffer.wrap() method and
named the class correspondingly.

Nah, ByteBuffer.wrap(byte[]) would return a ByteBuffer. This is reaching in and getting the implementation (i.e. almost the opposite).

We need to rename it.  How about InputStreamExposer#expose(InputStream).

Regards,
Tim

Reply via email to