develooper Front page | perl.perl5.porters | Postings from September 2014

[perl #122811] [PATCH] move assert_uft8_cache_coherent to DEBUGGING builds

Thread Next
From:
Father Chrysostomos via RT
Date:
September 19, 2014 23:56
Subject:
[perl #122811] [PATCH] move assert_uft8_cache_coherent to DEBUGGING builds
Message ID:
rt-4.0.18-2621-1411170958-1771.122811-15-0@perl.org
On Fri Sep 19 14:53:29 2014, bulk88 wrote:
> This is a bug report for perl from bulk88@hotmail.com,
> generated with the help of perlbug 1.40 running under perl 5.21.4.
> 
> 
> -----------------------------------------------------------------
> [Please describe your issue here]
> 
> See attached patch. I am not 100% sure about this patch, since people 
> did report bugs/tickets with these panic messages in the last couple of 
> years. The original intent does sound like it should be DEBUGGING code.
> 
> Google "panic sv_len_utf8" and "panic utf8_mg_pos_cache_update" for the 
> story, "panic sv_pos_b2u" return a couple very old hits too.

As you said, this has caught legitimate bugs in the past.  I think we should keep it in non-debugging builds for that reason.

If we were to make this change, then the ${^UTF8CACHE} entry in perlvar.pod would need to be updated.

-- 

Father Chrysostomos


---
via perlbug:  queue: perl5 status: new
https://rt.perl.org/Ticket/Display.html?id=122811

Thread Next


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