You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2013/03/28 10:57:53 UTC
Re: svn commit: r1461865 - in
/subversion/branches/fsfs-format7/subversion:
include/private/svn_packed_data.h include/svn_error_codes.h
libsvn_subr/packed_data.c
stefan2@apache.org wrote on Wed, Mar 27, 2013 at 21:53:02 -0000:
> Author: stefan2
> Date: Wed Mar 27 21:52:57 2013
> New Revision: 1461865
>
> URL: http://svn.apache.org/r1461865
> Log:
> On the fsfs-format7: Introduce a framework for space and time-
> efficient storage of binary data. For details, see the header
> file documentation.
>
We almost never use the 'SVN__' namespace. Shouldn't these bits be 'svn_packed__*'?
(This isn't nitpicking, this is a commit review that only reviews the
log message. :-) )
> * subversion/include/private/svn_packed_data.h
> (): new header
> (SVN__PACKED_DATA_BUFFER_SIZE): new constant
> (svn__packed_data_root_t,
> svn__packed_byte_stream_t,
> svn__packed_int_stream_t): new data types
> (svn__packed_data_create_root,
> svn__packed_create_int_stream,
> svn__packed_create_int_substream,
> svn__packed_create_bytes_stream,
> svn__packed_create_bytes_substream,
> svn__packed_data_flush_buffer,
> svn__packed_add_uint,
> svn__packed_add_int,
> svn__packed_add_bytes,
> svn__packed_data_write): declare data write API
> (svn__packed_first_int_stream,
> svn__packed_first_byte_stream,
> svn__packed_next_int_stream,
> svn__packed_next_byte_stream,
> svn__packed_first_int_substream,
> svn__packed_first_byte_substream,
> svn__packed_int_count,
> svn__packed_byte_count,
> svn__packed_data_fill_buffer,
> svn__packed_get_uint,
> svn__packed_get_int,
> svn__packed_get_bytes,
> svn__packed_data_read): declare data read API
>
> * subversion/include/svn_error_codes.h
> (SVN_ERR_CORRUPT_PACKED_DATA): define new error code
>
> * subversion/libsvn_subr/packed_data.c
> (): new source
> (packed_int_private_t): new private data type
> (svn__packed_byte_stream_t,
> svn__packed_data_root_t): define data types
> (create_bytes_stream_body,
> packed_data_create_bytes_substream_body,
> write_packed_uint_body,
> write_packed_uint,
> write_int_stream_structure,
> write_byte_stream_structure,
> write_stream_uint,
> packed_int_stream_length,
> packed_byte_stream_length,
> append_int_stream,
> append_byte_stream,
> write_stream_data,
> read_packed_uint_body,
> read_stream_uint,
> read_packed_uint,
> read_int_stream_structure,
> read_byte_stream_structure,
> read_stream_data,
> unflatten_int_stream,
> unflatten_byte_stream): new utility functions
> (svn__packed_data_create_root,
> svn__packed_create_int_stream,
> svn__packed_create_int_substream,
> svn__packed_create_bytes_stream,
> svn__packed_create_bytes_substream,
> svn__packed_data_flush_buffer,
> svn__packed_add_uint,
> svn__packed_add_int,
> svn__packed_add_bytes,
> svn__packed_data_write): implement data write API
> (svn__packed_first_int_stream,
> svn__packed_first_byte_stream,
> svn__packed_next_int_stream,
> svn__packed_next_byte_stream,
> svn__packed_first_int_substream,
> svn__packed_first_byte_substream,
> svn__packed_int_count,
> svn__packed_byte_count,
> svn__packed_data_fill_buffer,
> svn__packed_get_uint,
> svn__packed_get_int,
> svn__packed_get_bytes,
> svn__packed_data_read): implement data read API
>
> Added:
> subversion/branches/fsfs-format7/subversion/include/private/svn_packed_data.h
> subversion/branches/fsfs-format7/subversion/libsvn_subr/packed_data.c
> Modified:
> subversion/branches/fsfs-format7/subversion/include/svn_error_codes.h
Re: svn commit: r1461865 - in /subversion/branches/fsfs-format7/subversion:
include/private/svn_packed_data.h include/svn_error_codes.h libsvn_subr/packed_data.c
Posted by Stefan Fuhrmann <st...@wandisco.com>.
On Thu, Mar 28, 2013 at 10:57 AM, Daniel Shahaf <d....@daniel.shahaf.name>wrote:
> stefan2@apache.org wrote on Wed, Mar 27, 2013 at 21:53:02 -0000:
> > Author: stefan2
> > Date: Wed Mar 27 21:52:57 2013
> > New Revision: 1461865
> >
> > URL: http://svn.apache.org/r1461865
> > Log:
> > On the fsfs-format7: Introduce a framework for space and time-
> > efficient storage of binary data. For details, see the header
> > file documentation.
> >
>
> We almost never use the 'SVN__' namespace. Shouldn't these bits be
> 'svn_packed__*'?
>
Done in r1462837.
Thanks for the review!
-- Stefan^2.
--
*Join one of our free daily demo sessions on* *Scaling Subversion for the
Enterprise <http://www.wandisco.com/training/webinars>*
*
*
Re: svn commit: r1461865 - in /subversion/branches/fsfs-format7/subversion:
include/private/svn_packed_data.h include/svn_error_codes.h libsvn_subr/packed_data.c
Posted by Stefan Fuhrmann <st...@wandisco.com>.
On Thu, Mar 28, 2013 at 10:57 AM, Daniel Shahaf <d....@daniel.shahaf.name>wrote:
> stefan2@apache.org wrote on Wed, Mar 27, 2013 at 21:53:02 -0000:
> > Author: stefan2
> > Date: Wed Mar 27 21:52:57 2013
> > New Revision: 1461865
> >
> > URL: http://svn.apache.org/r1461865
> > Log:
> > On the fsfs-format7: Introduce a framework for space and time-
> > efficient storage of binary data. For details, see the header
> > file documentation.
> >
>
> We almost never use the 'SVN__' namespace. Shouldn't these bits be
> 'svn_packed__*'?
>
Done in r1462837.
Thanks for the review!
-- Stefan^2.
--
*Join one of our free daily demo sessions on* *Scaling Subversion for the
Enterprise <http://www.wandisco.com/training/webinars>*
*
*