On Mon, Aug 17, 2020 at 02:07:33PM +0200, Ruediger Pluem wrote: > On 11/21/19 4:51 PM, jor...@apache.org wrote: > > Author: jorton > > Date: Thu Nov 21 15:51:32 2019 > > New Revision: 1870095 > > > > URL: http://svn.apache.org/viewvc?rev=1870095&view=rev ... > > @@ -1132,6 +1144,17 @@ static int ssl_hook_Access_modern(reques > > } > > } > > > > + /* Fill reneg buffer if required. */ > > + if (change_vmode) { > > Why can't we put this block below the > > if (change_vmode) > > a few lines below?
Good question, thanks for the review. No good reason, and moving the call after the r->connection->master test is presumably better. Done in r1880927. Regards, Joe
