develooper Front page | perl.perl5.porters | Postings from January 2013

[perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_ret NULL check removal

From:
Tony Cook via RT
Date:
January 30, 2013 03:09
Subject:
[perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_ret NULL check removal
Message ID:
rt-3.6.HEAD-27190-1359515347-1611.116188-15-0@perl.org
On Tue Jan 29 03:19:44 2013, davem wrote:
> I agree. Regardless of any insignificant performance gains, the new code
> itself is slightly cleaner, and it's always good to declare function
args NN
> where possible; so I've a applied it as
> 
>     a03199eaa6f5d9e2d15c64750229e2adeebfbdce
> 
> but I rewrote the commit message to be clearer.

Unfortunately you messed up the author email address.

The attached patch has headers:

From 030ed137171a0d895b1b36f35c8a10c274a4535d Mon Sep 17 00:00:00 2001
From: Daniel Dragan <bulk88@hotmail.com>
Date: Sun, 23 Dec 2012 21:18:24 -0500

but I guess you <git am>ed the whole mailbox entry instead of the
attachment.

Tony




---
via perlbug:  queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=116188



nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About