[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#127005: package is broken on arm, powerpc and s390 due to wrong assumption on char signedness



Gerhard Tonn writes:

> Package: tetex-bin
> Version: 1.0.7+20011202-2
> Severity: important

> Hi,
> your package is broken on arm, powerpc and s390, since char is per default
> unsigned on these platforms. 

> Please check

> http://buildd.debian.org/build.php?&pkg=tetex-bin

> for details. Search for the following warnings

> comparison is always true due to limited range of data type 
> comparison is always false due to limited range of data type
> comparison of unsigned expression >= 0 is always true

These three are not necessarily bugs, though the code should be looked
over to be certain.

> initialization of negative value `-1' to `char'

Looks very bad.  Would look even worse if I'd manage to find an actual
occurrence in the reports.  Please supply more information (source
file & line, for example).

> pointer targets in assignment differ in signedness
> pointer targets in passing arg 1 of `...' differ in signedness

Same here.

> in the latest build log for s390.

> A fix for the problem is to change the type of the variable 
> shown by the warning to 'signed char'.

Don't do that until after you've inspected the code.


By the way, if you look in the IA64 logs you'll find messages that
indicate some real problem, I think in xdvik/oxdvik, pssoible
elsewhere as well.  It looked to me like the kind of symptoms you get
when a function returning pointers isn't declared, and the resulting
code will break on that platform.

-- 
Olaf Weber

               (This space left blank for technical reasons.)




Reply to: