Re: PATCH: redefine $decimal, $hex, $text vars

Jacob Bachmeyer <[email protected]>
Newsgroups gmane.comp.sysutils.dejagnu.general
Message-ID <[email protected]>
Ben Elliston wrote:
> I think this patch is an improvement becaue it (a) is much more
> readable and (b) matches non-ASCII characters when non-English locales
> are used.
>   

Are we sure that DejaGnu should be sensitive to locales at all?  Perhaps 
the runtest shell script should always export LOCALE=C?

I really do not think that the hex and decimal patterns should ever 
match non-ASCII:  those seem to be intended for parsing output from 
other programs.

Generally, matching non-ASCII characters in non-English locales sounds 
like a good idea at first glance, but I expect it to prove to be another 
"hidden influence" that users are likely to overlook.  We would need to 
be very careful about this, and changing variables that existing 
testsuites might depend on in this way is fraught with peril.  
(Annoyingly so, since I wanted to change "text" to {[^\r\n]+}.)


-- Jacob
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.