garydgregory commented on PR #784:
URL: https://github.com/apache/commons-io/pull/784#issuecomment-3321167669

   > ```java
   > InputStreamOrigin
   > ```
   
   This is definitely not a bug and is the expected behavior. A couple of 
points:
   - Maybe the type-specific origin classes like `InputStreamOrigin` shouldn't 
have been public, but it would be harder to provide new conversions outside the 
library itself, but maybe IO isn't used at that level of depth. A lesson 
learned for the future.
   - The type-specific origin classes (here, `InputStreamOriogin`) are all 
_wrappers_ of that specific type, and always return the instance it was given 
on creation from its same-type getter (here, `getInputStream()`). In this case, 
an `InputStreamOrigin` is constructed with an `InputStream` and always returns 
that `InputStream` from `getInputStream()`
   - The previous point highlights the need for better Javadocs I think.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to