Re: Patch: EMI UUCP DLRs (final)

2010-07-07 Thread Alexander Malysh
Hi, sorry for delay... I have limited inet access now... see answers bellow. Am 30.06.2010 um 19:30 schrieb Nikos Balkanas: > Hi, > > Please see inlined answers. > Thanks for the comments and corrections. Please confirm a few remaining > choices. > > BR, > Nikos > - Original Message -

Re: Patch: EMI UUCP DLRs (final)

2010-07-07 Thread Nikos Balkanas
Hi, This is a classical maintenance vs overhead. Malloc is expensive, much more so in Linux than in Solaris. Furthermore, sql="%s%S" is more difficult to read and understand, since SQL mechanism is not explicit, but hidden in variables. Do you really want that? yes, because maintenance is t

Re: smppbox code questions

2010-07-07 Thread Victor Luchitz
Any hope this will be reviewed and committed? I'm also working on a patch that adds TLV support to smppbox but I'd like to get this one included first. 2010/7/6 Victor Luchitz : > Yup, it's working fine now. Noticed there's another memleak though: > > another octstr_destroy(msgid); call is needed