[Devel] seas module

[Daniel-Constantin Mierla]

Got one more waring, please check:

encode_header.c: In function 'encode_header':
encode_header.c:214: warning: cast from pointer to integer of different size

Cheers,
Daniel



On 03/08/07 13:10, Daniel-Constantin Mierla wrote:
> Hello,
>
> On 03/08/07 13:05, Elias Baixas wrote:
>> Hi,
>>
>> just did a
>> find modules/seas/ -name "*.c" -o -name "*.h" |while read i; do sed 
>> "s/SLOG(/LOG(/g" $i > $i.2 && mv $i.2 $i; done
>>
>> and it compiled without complains... so it should also in SunOS and 
>> freeBSD V.whatever
> great, thanks.
>>
>> I'm fixing the warnings also (just casting to (int))... in what 
>> arch/OS where they seen ?
>
> was Debian etch amd64.
>
> Cheers,
> Daniel
>
>>
>> regards,
>>
>> Elias
>>
>> Daniel-Constantin Mierla wrote:
>>> Hello,
>>>
>>> apart (or related to) reported issues, there are warnings:
>>>
>>> ha.c: In function 'init_pingtable':
>>> ha.c:310: warning: format '%d' expects type 'int', but argument 5 
>>> has type 'long unsigned int'
>>> ...
>>>
>>> statistics.c: In function 'init_seas_stats_table':
>>> statistics.c:48: warning: format '%d' expects type 'int', but 
>>> argument 5 has type 'long unsigned int'
>>> ...
>>>
>>> If there will be no feedback by tomorrow, I will revert your changes 
>>> to custom logging system to a previous version. Not sure if along 
>>> with moving to this new logging mechanism there were other fixes, so 
>>> the alternative is to remove seas module from release 1.2.0.
>>>
>>> Cheers,
>>> Daniel
>>>
>>>
>>> On 03/06/07 14:44, Daniel-Constantin Mierla wrote:
>>>> Hello Elias,
>>>>
>>>> any update on this?
>>>>
>>>> Cheers,
>>>> Daniel
>>>>
>>>>
>>>> On 03/01/07 19:49, Daniel-Constantin Mierla wrote:
>>>>> Hello,
>>>>>
>>>>> as far as I see, the changes you introduced are not portable on 
>>>>> SunOS and other architecture ... the SVN is frozen for the next 
>>>>> release which is in about 1 week. Only fixes should be applied. We 
>>>>> have no time to port to all OSes right now.
>>>>>
>>>>> Was it so necessary to add these changes right now? For the next 
>>>>> future we have in mind to introduce options to get same kind of 
>>>>> log messages. I face right now same problem compiling on gg 
>>>>> 2.95/FreeBSD the SST module  which has as well a custom logging 
>>>>> system. Would be better to stick to the default on in openser, and 
>>>>> if it is limited, then we should improve that one, rather than 
>>>>> adding specific solutions for each module.
>>>>>
>>>>> Cheers,
>>>>> Daniel
>>>>>
>>>>>
>>>>> On 03/01/07 19:25, elias.baixas at voztele.com wrote:
>>>>>> Revision: 1725
>>>>>>           
>>>>>> http://openser.svn.sourceforge.net/openser/?rev=1725&view=rev
>>>>>> Author:   eliasbaixas
>>>>>> Date:     2007-03-01 09:25:55 -0800 (Thu, 01 Mar 2007)
>>>>>>
>>>>>> Log Message:
>>>>>> -----------
>>>>>> Changed the logging system
>>>>>>
>>>>>> Modified Paths:
>>>>>> --------------
>>>>>>     trunk/modules/seas/cluster.c
>>>>>>     trunk/modules/seas/encode_allow.c
>>>>>>     trunk/modules/seas/encode_contact.c
>>>>>>     trunk/modules/seas/encode_cseq.c
>>>>>>     trunk/modules/seas/encode_digest.c
>>>>>>     trunk/modules/seas/encode_header.c
>>>>>>     trunk/modules/seas/encode_msg.c
>>>>>>     trunk/modules/seas/encode_route.c
>>>>>>     trunk/modules/seas/encode_to_body.c
>>>>>>     trunk/modules/seas/encode_uri.c
>>>>>>     trunk/modules/seas/encode_via.c
>>>>>>     trunk/modules/seas/event_dispatcher.c
>>>>>>     trunk/modules/seas/ha.c
>>>>>>     trunk/modules/seas/seas.c
>>>>>>     trunk/modules/seas/seas.h
>>>>>>     trunk/modules/seas/seas_action.c
>>>>>>     trunk/modules/seas/seas_action.h
>>>>>>     trunk/modules/seas/statistics.c
>>>>>>     trunk/modules/seas/xaddress.h
>>>>>>
>>>>>>
>>>>>> This was sent by the SourceForge.net collaborative development 
>>>>>> platform, the world's largest Open Source development site.
>>>>>>
>>>>>> _______________________________________________
>>>>>> Devel mailing list
>>>>>> Devel at openser.org
>>>>>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>>>>>
>>>>>>   
>>>>>
>>>>> _______________________________________________
>>>>> Devel mailing list
>>>>> Devel at openser.org
>>>>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>>>>
>>>>
>>>> _______________________________________________
>>>> Devel mailing list
>>>> Devel at openser.org
>>>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>>>
>>>
>>
>>
>
> _______________________________________________
> Devel mailing list
> Devel at openser.org
> http://openser.org/cgi-bin/mailman/listinfo/devel
>