Rename svn_stream_buffered() and related symbols to ...is_buffered... for clarity.
* subversion/include/svn_io.h (svn_io_buffered_fn_t): Rename to svn_io_is_buffered_fn_t. (svn_stream_set_buffered): Rename to svn_stream_set_is_buffered. (svn_stream_buffered): Rename to svn_stream_is_buffered.
* subversion/libsvn_subr/stream.c (svn_stream_t): Rename 'buffered_fn' to 'is_buffered_fn'. (svn_stream_set_buffered): Rename to svn_stream_set_is_buffered. (svn_stream_buffered): Rename to svn_stream_is_buffered. (svn_stream_create, stream_readline, svn_stream_empty, buffered_handler_disown, svn_stream_disown, svn_stream_from_aprfile2, svn_stream_from_stringbuf, svn_stream_from_string): Update calls.
* subversion/libsvn_subr/subst.c (translated_stream_buffered): Rename to translated_stream_is_buffered. (stream_translated): Update calls.
This minor change to translate_chunk() reduces the checkout time of ^/subversion/trunk/subversion in my profiler by about 1% for me. If this has the same result on production code then this patch will probably safe quite a few trees.
This part of the code was imported from stefan2's performance branch. More profiling results (especially from stefan2) welcome.
* subversion/libsvn_subr/subst.c (translate_chunk): Lose temporary variable and use boolean operators instead of bitwise.