[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [Patch] (swig-py) accept core.svn_stream_t object for svn_stream_t *

From: Michael Pilato <cmpilato_at_collab.net>
Date: Mon, 10 Dec 2018 12:51:26 +0000

On 12/10/18 7:03 AM, Branko Čibej wrote:
> On 09.12.2018 17:52, Yasuhito FUTATSUKI wrote:
>> Ah, former patch has already been commited. Thank you.
>> I'm sorry I didn't watch trunk.
>
> That's fine! And thank you for your contribution.
>
> If you think your second patch is better, please consider updating it so
> that it applies to current trunk.
>
> -- Brane
>

Agreed! In fact, I was actually trying to take your first patch and
make the same changes present in the second (because I noticed while
grepping for something that your large block of added code was getting
replicated dozens of times across the generated .c files). But
mentally, I wasn't top-notch on Friday and ended up bailing on my
attempt and committing your patch as-is instead.

As punishment, I'll get the updated patch reviewed and committed.

-- Mike
Received on 2018-12-10 13:51:45 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.