[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[avrdude-dev] [bug #39794] warnings when building avrdude 6.0rc1 under C
From: |
Galen Seitz |
Subject: |
[avrdude-dev] [bug #39794] warnings when building avrdude 6.0rc1 under CentOS 6.4 |
Date: |
Fri, 16 Aug 2013 04:34:34 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130807 Firefox/17.0 |
URL:
<http://savannah.nongnu.org/bugs/?39794>
Summary: warnings when building avrdude 6.0rc1 under CentOS
6.4
Project: AVR Downloader/UploaDEr
Submitted by: galens
Submitted on: Fri 16 Aug 2013 04:34:34 AM GMT
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
Item Group: None
Status: None
Privacy: Public
Assigned to: None
Originator Name: Galen Seitz
Originator Email:
Open/Closed: Open
Discussion Lock: Any
_______________________________________________________
Details:
While building avrdude 6.0rc1 under CentOS 6.4, I encountered the following
warnings. I posted these to the avrdude-dev mailing list, and Joerg
responded. I have included some of Joerg's response.
gcc -DHAVE_CONFIG_H -I. -DCONFIG_DIR=\"/usr/local/etc\" -Wall
-Wno-pointer-sign -g -O2 -MT libavrdude_a-avr.o -MD -MP -MF
.deps/libavrdude_a-avr.Tpo -c -o lib avrdude_a-avr.o `test -f 'avr.c' || echo
'./'`avr.c
avr.c: In function 'avr_tpi_program_enable':
avr.c:131: warning: passing argument 2 of 'pgm->cmd_tpi' discards qualifiers
from pointer target type
avr.c:131: note: expected 'unsigned char *' but argument is of type 'const
unsigned char *'
Joerg> This means argument 2 of the cmd_tpi method should be made expect a
pointer to const. Not a big deal, but a number of files have to be changed.
gcc -DHAVE_CONFIG_H -I. -DCONFIG_DIR=\"/usr/local/etc\" -Wall
-Wno-pointer-sign -g -O2 -MT libavrdude_a-fileio.o -MD -MP -MF
.deps/libavrdude_afileio.Tpo -c -o libavrdude_a-fileio.o `test -f 'fileio.c'
|| echo './'`fileio.c
fileio.c: In function 'elf2b':
fileio.c:947: warning: 'elf_getshstrndx' is deprecated (declared at
/usr/include/libelf.h:281)
Joerg> I already changed that (replaced by elf_getshdrstrndx()).
fileio.c:967: warning: format '%d' expects type 'int', but argument 4 has type
'size_t'
fileio.c:1027: warning: cast from pointer to integer of different size
fileio.c:1027: warning: format '%d' expects type 'int', but argument 5 has
type 'size_t'
fileio.c:1057: warning: format '%d' expects type 'int', but argument 3 has
type 'size_t'
Joerg> That's difficult. C99 declares a new printf() format for size_t ("z"),
but last time we've been there, Cygwin didn't have it.
gcc -DHAVE_CONFIG_H -I. -DCONFIG_DIR=\"/usr/local/etc\" -Wall
-Wno-pointer-sign -g -O2 -MT libavrdude_a-jtag3.o -MD -MP -MF
.deps/libavrdude_a-jtag3.Tpo -c -o libavrdude_a-jtag3.o `test -f 'jtag3.c' ||
echo './'`jtag3.c
jtag3.c: In function 'jtag3_initialize':
jtag3.c:850: warning: 'flashsize' may be used uninitialized in this function
Joerg> Should not trigger in practice, but the compiler cannot know this. (It
would remain uninitialized indeed if someone created a config entry that has
no "flash" section.)
gcc -DHAVE_CONFIG_H -I. -DCONFIG_DIR=\"/usr/local/etc\" -Wall
-Wno-pointer-sign -g -O2 -MT libavrdude_a-pickit2.o -MD -MP -MF
.deps/libavrdude_a-pickit2.Tpo -c -o libavrdude_a-pickit2.o `test -f
'pickit2.c' || echo './'`pickit2.c
pickit2.c: In function 'usb_open_device':
pickit2.c:1170: warning: cast from pointer to integer of different size
Joerg> It should probably print the pointer using the %p format.
Here are the versions of the compiler and two of the related libraries.
$ rpm -q gcc
gcc-4.4.7-3.el6.x86_64
$ rpm -q elfutils-libelf
elfutils-libelf-0.152-1.el6.x86_64
$ rpm -q libusb
libusb-0.1.12-23.el6.x86_64
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/bugs/?39794>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [avrdude-dev] [bug #39794] warnings when building avrdude 6.0rc1 under CentOS 6.4,
Galen Seitz <=