RE: compiler warnings
"Angel Fradejas" <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <[email protected]> |
size_t looks good to me. Angel Fradejas Mediafusion Espana, S.A. [email protected] www.mediafusion.es Tel. +34 91 252 32 00 Fax +34 91 572 27 08 -----Mensaje original----- De: [email protected] [mailto:[email protected]]En nombre de Harrie Hazewinkel Enviado el: miercoles 31 de julio de 2002 11:20 Para: dev-kannel Asunto: Re: compiler warnings HI, --On Wednesday, July 31, 2002 12:53 AM +0100 Bruno David Rodrigues <[email protected]> wrote: > On Ter, 2002-07-30 at 10:07, Harrie Hazewinkel wrote: >> HI, >> >> I noticed that Stipe fixed compiler warnings after my patch. >> So I thought using '--enable-warnings' to have '-Wall' as compile >> option, but I would like to add additional warnings like >> '-Wmissing-prototypes' and '-Wmissing-declarations'. > > Yes, please, add every -W* required to full debug the compile process. > I've added --enable-warnings to be able to use the -Wall everytime I > make "make" > While adding the various -W* warnings I stumble accros various warnings. I am trying to fix those. But the main one is a signed/unsigned conversion. I believe we better fix these. So, for instance, length and size variable should become all unsigned I could take either unsigned long or size_t and would prefer to use size_t for both is this OK with you all?? Harrie Internet Management Consulting mailto:[email protected] http ://www.mod-snmp.com/ ------------------------------------------------------------------- Author of MOD-SNMP, enabling SNMP management to the Apache server.