Re: svn commit: r1683521 - /apr/apr/trunk/network_io/unix/sockaddr.c
Am 25.10.2017 um 15:21 schrieb Joe Orton: On Wed, Oct 25, 2017 at 10:22:46AM +0200, Rainer Jung wrote: Before the change, copying starts at "scope_delim + 1", after the change at "scope_delim". The log says "Simplify to use apr_pstrmemdup, no functional change.", so it seems that was not an intentional change. Shouldn't we correct it with the following change: How annoying, sorry :( Yes you're obviously right. NP, thanks a bunch for your prompt reaction! Rainer
buildbot success in on apr-trunk
The Buildbot has detected a restored build on builder apr-trunk while building . Full details are available at: https://ci.apache.org/builders/apr-trunk/builds/173 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: orcus_ubuntu Build Reason: The AnyBranchScheduler scheduler named 'on-apr-commit' triggered this build Build Source Stamp: [branch apr/apr/trunk] 1813286 Blamelist: jorton Build succeeded! Sincerely, -The Buildbot
Re: svn commit: r1683521 - /apr/apr/trunk/network_io/unix/sockaddr.c
On Wed, Oct 25, 2017 at 10:22:46AM +0200, Rainer Jung wrote: > Before the change, copying starts at "scope_delim + 1", after the change at > "scope_delim". The log says "Simplify to use apr_pstrmemdup, no functional > change.", so it seems that was not an intentional change. Shouldn't we > correct it with the following change: How annoying, sorry :( Yes you're obviously right. http://svn.apache.org/viewvc?view=revision=1813286