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.


Reply via email to