On 21/06/18 01:08, Evgeny Kotkov wrote: > William A Rowe Jr <wr...@rowe-clan.net> writes: > >> Confusing... which patch is missing? >> >> http://svn.apache.org/viewvc?view=revision&revision=1808457 >> http://svn.apache.org/viewvc?view=revision&revision=1829962 >> >> It should not be possible at this point to need 1.x specific code >> that wouldn't correspond to a trunk commit. > > I think that r1829962 should be treated as a follow-up to r1808457 > (where I missed one calling site of apr_thread_mutex_unlock() in buffer.c). > > In other words, they should either be backported together, or not backported > at all. Since this change is a relatively minor optimization (not a fix), > I would think that it might make sense to keep it just in trunk.
Actually it is a fix logresolve cores without it ;-) OK if everyone agrees I will just commit my small fix and don't backport the 2 other revisions. Cheers Jean-Frederic > > > Thanks, > Evgeny Kotkov >