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

On 01/25/2013 09:43 PM, Andy Dougherty wrote:
> So I'm not sure if it's worth applying or not.  What do you think?

All of this just my opinion: Unless there's a particular reason 
(maintainability, etc) not to, yes, worth applying. Removing a branching 
instruction by itself wouldn't be measurable at a macroscopic level 
unless it's in extraordinarily hot code. But doing so in many place 
eventually should be.

--Steffen

• [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_ret NULL check removal by bulk88
• Re: [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN andS_hv_free_ent_ret NULL check removal by Andy Dougherty
• Re: [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_retNULL check removal by Steffen Mueller
• Re: [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN andS_hv_free_ent_ret NULL check removal by Dave Mitchell
• [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_ret NULL check removal by Tony Cook via RT
• [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_ret NULL check removal by bulk88 via RT
• Re: git-hook to prevent perlbug as patch author by Steffen Mueller
• Re: git-hook to prevent perlbug as patch author by Tony Cook
• Re: git-hook to prevent perlbug as patch author by Tony Cook
• Re: git-hook to prevent perlbug as patch author by Tony Cook
• Re: git-hook to prevent perlbug as patch author by Steffen Mueller
• Re: git-hook to prevent perlbug as patch author by Craig A. Berry
• Re: git-hook to prevent perlbug as patch author by Thomas Sibley
• Re: git-hook to prevent perlbug as patch author by Steffen Mueller
• Re: git-hook to prevent perlbug as patch author by Tony Cook
• Re: git-hook to prevent perlbug as patch author by Tony Cook
• Re: git-hook to prevent perlbug as patch author by Steffen Mueller
• [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN and S_hv_free_ent_ret NULL check removal by bulk88 via RT
• Re: [perl #116188] [PATCH] hv.c: add some SvREFCNT_dec_NN andS_hv_free_ent_ret NULL check removal by Andy Dougherty
• Smoke [blead] v5.25.6-268-g15b6d28 FAIL(X) os/390 25.00 (2964/) by Yaroslav Kuzmin
• Re: Smoke [blead] v5.25.6-268-g15b6d28 FAIL(X) os/390 25.00 (2964/) by Karl Williamson