sanoopsivan / jsmpp

Automatically exported from code.google.com/p/jsmpp
Apache License 2.0
0 stars 0 forks source link

Optional parameter support #1

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hi, 

We're testing jsmpp for usage in our application, and came across the need
to support optional parameters in PDUs.

The attached patch contains a simple implementation, allowing:
* Adding OptionalParameters to PDU's
* Serialising and deserialising the parameters

Patch created against HEAD, r42.

Original issue reported on code.google.com by mikko.ko...@gmail.com on 28 Feb 2008 at 8:50

Attachments:

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Int param serialisation had a typo, and Byte params didn't get serialised at 
all.
This patch contains the previous content and these fixes.

Original comment by mikko.ko...@gmail.com on 7 Mar 2008 at 10:12

Attachments:

GoogleCodeExporter commented 8 years ago
Hi mikko, 

Cool and nice.
I will consider and add this feature. Previously I didn't add this feature 
because I didn't need it at the first time. 
Thanks a lot.

Original comment by uuda...@gmail.com on 8 Mar 2008 at 5:22

GoogleCodeExporter commented 8 years ago
This feature already implemented in version 2.x

Original comment by uuda...@gmail.com on 8 May 2008 at 5:08

GoogleCodeExporter commented 8 years ago
Done

Original comment by uuda...@gmail.com on 4 Jun 2008 at 3:55