Re: CVS update: subversion/subversion/include svn_delta.h

On Wed, Feb 28, 2001 at 11:06:19PM -0000, sussman@tigris.org wrote:
>...
> /* Set *STREAM to a pointer to a delta stream that will turn an empty
> - bytestream into STRING. */
> + bytestream into STRING. Do necessary allocation in POOL.
> +
> + When we call `svn_txdelta_next_window' on *STREAM, the window
> + returned will contain pointers directly into STRING (rather than
> + duplicating STRING's data.) So callers beware: make sure STRING
> + is not freed before STREAM and its windows! */
> void svn_txdelta_from_string (svn_txdelta_stream_t **stream,
> svn_string_t *string,
> apr_pool_t *pool);

That first sentence doesn't make sense. What does "empty bytestream" have to
do with anything? And we aren't putting anything "into" STRING, are we?

It seems more like this is a way to *read* a txdelta stream *from* a string,
right?

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/