svn commit: r243474 - head/usr.bin/cut

Andrey Chernov ache at freebsd.org
Sat Nov 24 14:51:50 UTC 2012


On 24.11.2012 18:12, Dimitry Andric wrote:
>>> -is_delim(int ch)
>>> +is_delim(wchar_t ch)
>>>   {
>>>       if (wflag) {
>>>           if (ch == ' ' || ch == '\t')
>>>
>>
>> I can't look at the whole code at this moment, but taking standalone
>> this is incorrect comparison for wchar_t. Should be L' ' and L'\t'
>> instead.
> 
> The compiler just promotes the space and tab to wchar_t, this is only
> needed for the sake of consistency (and/or style).  There is no binary
> change if you add the L prefixes. :)

In general case byte order of type promotion is not necessary equal to
byte order of L'x' literals.



More information about the svn-src-all mailing list