On 04.09.2015 21:17, stef...@apache.org wrote: > Author: stefan2 > Date: Fri Sep 4 19:17:44 2015 > New Revision: 1701317 > > URL: http://svn.apache.org/r1701317 > Log: > Finally, make svn_ra_svn__list_t actually a fully typed, ra_svn-specific > object. Update the creation functions; everything else already "just fits".
How is this code different from using APR arrays, except that the latter needs a typecast on array item access? As far as I can see, you've completely duplicated the APR array allocation strategy, including using two allocations to create the array. The only significant difference is that capacity is being tracked outside the svn_ra_svn__list_t structure during the construction of the list. Call me dense ... but can you please explain how exactly is this better/faster than using APR arrays? (I'm not going to mention 'safer' because it clearly isn't.) Code like this that is apparently meant be an optimisation of something(?) really should have a bit of an explanatory comment, IMO. -- Brane
