From: John Maddock (john_at_[hidden])
Date: 2008-07-01 04:58:30
Beman Dawes wrote:
>> John Maddock wrote:
>>>
>>> Would it be possible for the inspection report to print out the line
>>> containing the non-ASCII characters? There are a few files that are
>>> being flagged up, where I just can't find anything wrong with them
>>> :-(
>>
>> Hum... Take a look at trunk\tools\inspect\ascii_check.cpp
>>
>> It seems misnamed; it is apparently really checking for characters
>> the
>> c++ standard says are OK in source programs, regardless of encoding.
>>
>> Also, I notice the code:
>>
>> if ( c >= 'a' && c <= 'z' ) return false;
>> if ( c >= 'A' && c <= 'Z' ) return false;
>>
>> That isn't right for EBCDIC. See http://en.wikipedia.org/wiki/EBCDIC.
>> Although that is being pedantic - there is little chance the code
>> will
>> ever run on an non-ASCII system.
>>
>> But before changing anything, we really need to figure out what our
>> Boost standard is. How about anything 0x20-0x7E plus 0x09, 0x0A,
>> 0x0D?
>>
>> (0x09 is a tab, but we already have a more specific check for that.)
I'm all for strict checking, but at present it's too hard to find out what's
wrong :-(
Does the character set in section 2.2 apply to the contents of strings as
well as code BTW? If so then the strict checks may be justified...?
John.