Re: svn commit: r1683521 - /apr/apr/trunk/network_io/unix/sockaddr.c

2017-10-25 Thread Rainer Jung

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

2017-10-25 Thread buildbot
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

2017-10-25 Thread 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.

http://svn.apache.org/viewvc?view=revision=1813286