search

kucci / guava-libraries

Automatically exported from code.google.com/p/guava-libraries
Apache License 2.0
0 stars 0 forks source link

Plain parameters API in CharStreams #452

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
CharStreams has methods with InputStreamSupplier.
In would be great if such methods would be overloaded with methods that take 
only InputStream as a parameter.

e.g., CharStreams has toString(InputStreamSupplier supplier)
One needs to add toString(InputStream inputStream)

Original issue reported on code.google.com by nordlig....@gmail.com on 18 Oct 2010 at 7:44

GoogleCodeExporter commented 9 years ago 
See this discussion group thread:

http://groups.google.com/group/guava-discuss/browse_thread/thread/88976574f28394
b0/e46fad488792f084

Original comment by fin...@gmail.com on 1 Nov 2010 at 3:12

GoogleCodeExporter commented 9 years ago 
It tells nothing about InputStreamSupplier vs plain InputStream parameter

Original comment by nordlig....@gmail.com on 1 Nov 2010 at 9:44

GoogleCodeExporter commented 9 years ago 
I think he meant this one: 
http://groups.google.com/group/guava-discuss/browse_thread/thread/eecc5ce04bbeb9
5/a81f83e0758e0630?lnk=gst&q=InputSupplier#a81f83e0758e0630. It, along with the 
documentation associate with InputSupplier, makes an interesting case for why 
they did stuff the way they did.

Depending on your needs, you may be able to get a decent amount of 
functionality using CharStreams#copy.

Original comment by bryan.de...@gmail.com on 11 May 2011 at 12:46

GoogleCodeExporter commented 9 years ago

Original comment by kevinb@google.com on 18 Jul 2011 at 3:41

GoogleCodeExporter commented 9 years ago 
This issue has been migrated to GitHub.

It can be found at https://github.com/google/guava/issues/<id>

Original comment by cgdecker@google.com on 1 Nov 2014 at 4:15

GoogleCodeExporter commented 9 years ago

Original comment by cgdecker@google.com on 3 Nov 2014 at 9:09