Optional parameter support

[GoogleCodeExporter]

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:

• optional-parameters.patch

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

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:

• optional-parameters-2.patch

[GoogleCodeExporter]

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

• Changed state: Accepted

[GoogleCodeExporter]

This feature already implemented in version 2.x

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

• Changed state: Started

[GoogleCodeExporter]

Done

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

• Changed state: Fixed