Printf format warnings

Bug #237077 reported by Michael D. Adams
2
Affects Status Importance Assigned to Milestone
Ikarus Scheme
Fix Committed
Low
Abdulaziz Ghuloum

Bug Description

The printf format "%d" should be changed to "%ld" when the argument is a "long int" rather than just an "int". Also the format "%x" expects an "unsigned int" while "%lx" expects an "unsigned long int".

The following places need changes to reflect that. At present they generate warnings under "make CFLAGS='-W -Wall'".

src/ikarus-collect.c line 506: change "%d" to "%ld"
src/ikarus-collect.c line 1147: change "%08x" to "%08lx"
src/ikarus-collect.c line 1190: change "long int i" to "unsigned long int i"
src/ikarus-numerics.c line 51: change "%08x" to "%08lx" and change "(long int)fst" to "fst" since fst is already "unsigned long int".

Related branches

Revision history for this message
Abdulaziz Ghuloum (aghuloum) wrote :

Fixed *all* "-Wall" warnings in revision 1495. Thanks.

Changed in ikarus:
assignee: nobody → aghuloum
status: New → Fix Committed
importance: Undecided → Low
Changed in ikarus:
milestone: none → 0.0.4
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.