On Thu, Aug 20, 2015 at 11:16 AM, nwellnhof <[email protected]> wrote:
> GitHub user nwellnhof opened a pull request:
>
>     https://github.com/apache/lucy-clownfish/pull/36
>
>     Rework Perl SV to Clownfish object conversion

I reviewed this, finally.  Changing up both refcounts and nulls is
really tricky, but I looked at each call site in both Clownfish and
Lucy and I couldn't find any mistakes.  Yowza!

I'm a little surprised that we didn't need the combination of nullable
and noinc.  But it's rare and can be faked by with nullable conversion
followed by a DECREF.

Marvin Humphrey

Reply via email to