Bringing this portion from [1] back to this thread: On Sun, May 24, 2020 at 8:38 PM Daniel Shahaf <d...@daniel.shahaf.name> wrote: > Nathan Hartman wrote on Sun, 24 May 2020 18:28 -0400: > > Asking because I ran the latter on latest trunk with this change > > applied. (All tests passed.) Also from reading the code, including > > spelunking into the APR calls etc, I see no functional difference > > (except efficiency as stated). > > "except efficiency"?
:-) I meant: I see no functional difference, except that this implementation should construct the array more efficiently than the previous one. Nathan [1] "Sporadic davautocheck failure in trunk? (was: Re: [PATCH] Fix an inefficient way to fill an array of inherited properties)" 24 May 2020 https://lists.apache.org/thread.html/r7e332a8d760527d754d4a5d23094d7712605a4eb10e76a8e91f1f2c5%40%3Cdev.subversion.apache.org%3E