Re: svn commit: r1616600 - in /subversion/trunk/subversion: include/svn_error_codes.h libsvn_fs_x/pack.c libsvn_fs_x/reps.c libsvn_fs_x/reps.h tests/libsvn_fs_x/fs-x-pack-test.c

On 07.08.2014 23:35, stefan2_at_apache.org wrote:
> Author: stefan2
> Date: Thu Aug 7 21:35:01 2014
> New Revision: 1616600
>
> URL: http://svn.apache.org/r1616600
> Log:
> Explicitly limit the dimensions of a star delta container which allows us
> to cast sizes to 32 bits.

Can we explicitly use 32-bit types then, pretty please? Peppering the
code with casts is not the way to silence compiler warnings. Using
correct types is.

That might require adding number pack/unpack functions that work with
correctly sized types, but since the idea is to detect overflow in the
numbers, this is actually a good idea (and does not affect the packed
representation of the numbers).

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco | Realising the impossibilities of Big Data
e. brane_at_wandisco.com