Re: svn commit: r39635 - in trunk/subversion/bindings/swig: . include python/libsvn_swig_py

Stefan Sperling wrote:
> > + APR_ARRAY_IDX(temp, targlen, void *) = struct_ptr;
>
> You could use APR_ARRAY_PUSH(temp, void *) = struct_ptr; here.
> That would align better with common use of arrays in our code,
> PUSH is usually used for writing to the array, IDX for retrieval.

No, PUSH is different: it means extend the array by one element and then
set that element.

- Julian

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2402614
Received on 2009-10-01 19:12:18 CEST