Stefan Sperling <s...@elego.de> writes: > On Fri, Apr 16, 2010 at 11:38:41AM +0100, Philip Martin wrote: >> Julian Foad <julian.f...@wandisco.com> writes: >> > Just write "arr->nelts = 0;" (which is the entire body of >> > apr_array_clear()) and don't introduce an ifdef. > > I'm a bit sceptical. > Do we really want to rely on implementation details like > "arr->nelts = 0" in the APR code?
Two examples where we already set nelts: $ grep nelts\ =[^=] ../src/subversion/libsvn_*/*c ../src/subversion/libsvn_fs_base/dag.c: copy_ids->nelts = i_out; ../src/subversion/libsvn_fs_fs/caching.c: manifest->nelts = data_len / sizeof(apr_off_t); -- Philip
