develooper Front page | perl.perl5.porters | Postings from August 2010

[perl #76938] PATCH: charnames regression: fails when no import ever done

Thread Next
From:
karl williamson
Date:
August 2, 2010 16:38
Subject:
[perl #76938] PATCH: charnames regression: fails when no import ever done
Message ID:
rt-3.6.HEAD-2463-1280792259-1375.76938-75-0@perl.org
# New Ticket Created by  karl williamson 
# Please include the string:  [perl #76938]
# in the subject line of all future correspondence about this issue. 
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=76938 >


There is a bug in the last patch I made to charnames which introduces a 
regression, when all charnames calls in a program are of the form 'use 
charnames ()'.  The bug is only in blead.

This series of commits fixes that bug, and another in the test file, in 
which an empty line got added to the end of the .t.  git diff doesn't 
detect it; which I guess is a bug in that (perhaps fixed in a later 
version than I'm running).  The blank line got interpreted as 
overwriting the name for NULL with the empty string.  I don't know if 
the blank line is in what I committed, or what is already in blead, but 
I added code to skip all such lines.  Note to committer: check and 
remove any empty line at the end of the file.

Also, I converted the .t to use srand's seed, instead of computing it 
itself, and use fewer tests of algorithmically determined names.

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