Re: [PATCH] Fix an inefficient way to fill an array of inherited properties
On 24.05.2020 00:17, Daniel Shahaf wrote:
> Branko Čibej wrote on Fri, 15 May 2020 17:07 +0200:
>> On 15.05.2020 16:54, Denis Kovalchuk wrote:
>>> I think that currently the code that obtains inherited properties for servers
>>> that don't support the capability uses an inefficient way of array filling.
>>> At each iteration, all previously added properties are moved one position to
>>> the right to maintain depth-first order. I think it's possible to provide a
>>> similar behavior walking the array of iprop requests from the end and get rid
>>> of extra copying of array elements.
>>> I've attached a patch.
>> Looks like an obvious fix. +1
> Any reason this hasn't been committed?
For my part, I just didn't get around to it. I think it should be committed.
Received on 2020-05-24 02:38:04 CEST
This is an archived mail posted to the Subversion Dev