Yes that how it is currently, just semantically I think it would be better to have all the PDU assembly/disassembly functions in one place.
It may also benefit others in future to have compatible pack/unpack functions out the box. On Saturday, 24 September 2016, Rene Kluwen <[email protected]> wrote: > Sure, but why not change KSMPPD to use smpp_pdu_pack() function without > the length? > > > > == Rene > > > > > > *Van:* devel [mailto:[email protected] > <javascript:_e(%7B%7D,'cvml','[email protected]');>] *Namens *Donald > Jackson > *Verzonden:* zaterdag 24 september 2016 12:07 > *Aan:* kannel_dev_mailinglist <[email protected] > <javascript:_e(%7B%7D,'cvml','[email protected]');>> > *Onderwerp:* [PATCH] KSMPPD (and other?) utility methods for smpp_pdu.c > > > > Hi all, > > > > Currently KSMPPD runs a patch against Kannel SVN trunk, I believe it would > be better if we could just include this patch in the mainline Kannel as > they don't have any negative implications. > > > > Currently the smpp_pdu_unpack() function takes data without length but the > smpp_pdu_pack() function provides length so these two functions are not > compatible with each other. This simple patch just allows for > compatibility convenience. > > > > Thanks, > > Donald > -- Donald Jackson http://www.elite-sms-software.com http://www.ddj.co.za http://www.thearchitech.com donald(a)thearchitech.com
