Oded Arbel wrote:
> 
> * smsc_id - if NULL, it duplicates the conn->name, but it does so before
> conn->name is created. not nice :-)

could you provide a quick patch for how you mean it would be looking
better, please?

> * destruction of relation_UTC_time in msg_to_pdu() - checking if it is
> null isn't needed, since octstr_destroy checks for it to be !NULL
> before destroying it.

yep, you're right. I added both declarative initialization and the
NULL checking, which is done inside octstr_destroy() itself. Removing
this.

> * in computing relation_UTC_time, the logic assumes that all time zones
> have differences of 1 hour to one another, which is not correct - there
> are some timezones that have a difference of half an hour.

ups, Alan, can you provide a fix for this?

Stipe

[EMAIL PROTECTED]
-------------------------------------------------------------------
Wapme Systems AG

M�nsterstr. 248
40470 D�sseldorf

Tel: +49-211-74845-0
Fax: +49-211-74845-299

E-Mail: [EMAIL PROTECTED]
Internet: http://www.wapme-systems.de
-------------------------------------------------------------------
wapme.net - wherever you are

Reply via email to