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

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

Thread Next
Father Chrysostomos via RT
September 19, 2014 23:56
[perl #122811] [PATCH] move assert_uft8_cache_coherent to DEBUGGING builds
Message ID:
On Fri Sep 19 14:53:29 2014, bulk88 wrote:
> This is a bug report for perl from,
> 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

Thread Next Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at | Group listing | About