Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 24 Nov 2012 12:00:54 +0400
From:      Andrey Chernov <ache@freebsd.org>
To:        Andrew Turner <andrew@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r243474 - head/usr.bin/cut
Message-ID:  <50B07EB6.3010503@freebsd.org>
In-Reply-To: <201211240415.qAO4FPaG062952@svn.freebsd.org>
References:  <201211240415.qAO4FPaG062952@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 24.11.2012 8:15, Andrew Turner wrote:
>   The is_delim function works on wchar_t characters not ints, update the
>   function to take a wchar_t as it's argument.
>  static int
> -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.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?50B07EB6.3010503>