On Wed, Oct 7, 2015 at 3:02 PM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
> On 7 October 2015 at 15:57, Stefan Fuhrmann
> <stefan.fuhrmann_at_wandisco.com> wrote:
> > On Wed, Oct 7, 2015 at 2:47 PM, Julian Foad <julianfoad_at_gmail.com>
> >> Stefan wrote:
> >> > I guess the correct way of doing this is revert Ivan's
> >> > change and apply something like the attached patch.
> >> Ivan wrote:
> >> > Here is the patch that I wanted commit later. What do you think?
> >> I am not interested in reviewing any more patches that tweak just one
> >> of the implementations. If you (anybody) want to do something, please
> >> combine the two functions into one implementation, and then I will be
> >> interested in reviewing the (single) implementation.
> > Alright, here you go.
> Hi Stefan,
> Are you sure that code in proposed patch compiles at least? :)
It does, GCC is quite lenient when it comes to pointer types ;)
But it has caused a few test failures. Here the final version.
If that doesn't work either then I'm done for today.
Received on 2015-10-07 15:17:03 CEST