[pacman-dev] [PATCH 2/3] pacsort, introduce define for escape_char error code
Allan McRae
allan at archlinux.org
Mon Jan 4 04:08:00 UTC 2016
On 31/12/15 23:19, Rikard Falkeborn wrote:
> The signedness of char is implementation defined. On systems where
> char is unsigned, comparing a variable of type char with -1 is never
> true, due to integer promotion rules. To avoid this, introduce a
> define for invalid field separators where -1 is cast to char. This will
> ensure that the return value check works for both unsigned and signed char.
>
> Fixes one warning [-Wtype-limits] for comparissons with -1 when compiling
> with -funsigned-char.
>
> Signed-off-by: Rikard Falkeborn <rikard.falkeborn at gmail.com>
OK.
More information about the pacman-dev
mailing list