Stefan Sperling wrote:
On Tue, Jan 16, 2018 at 07:37:31AM +0100, Branko Čibej wrote:
2. What's wrong with using svn_stream_readline() with
svn_io_stream_for_stdin2()?
In other words: I suspect this new function should be removed.
[...]
This function is in fact just a convenience wrapper around the two other
functions you mention.
There are 7 callers of this function across our command line tools.
If this function is not deemed appropriate for the public API, I would just
move it to the libsvn_subr cmdline.c bag of helper functions. Agreed?
That is also the thought that came to me overnight.
Renamed and moved to svn_cmdline__stdin_readline() in r1821224.
Thanks.
- Julian
