[OpenSER-Devel] Request for comments: sdpparser: module or integrated into the core

[Ovidiu Sas]

Hi Bogdan,


I started to play with the functions that are already implemented by
the nathelper.

We have two options here:
1. integrate into the core:
  - advantages:
    - easy to maintain, access, reuse the existing parsing infrastructure
  - disadvantage:
    - the core footprint will grow (not everyone will use the parser)

2. create a new module:
  - advantages:
    - only the configurations that need the sdp parser will load the module
    - small footprint for the core
  - disadvantages
    - difficult memory management if two modules are trying to use the parser



I would like to go with option one and integrate everything under the
parser directory.



The idea is to create some structure like this and link them:

sdp -> multiple sessions
session -> multiple streams
stream -> multiple payloads

each sdp has:
  - number of sessions: /* unsigned int: no_of_sessions */

each session has:
  - number of streams /* unsigned int: no_of_streams */

each stream has:
  - one IP address, /* str: 1.2.3.4 */
  - one port, /* str: 12345 */
  - one media type, /* str: audio */
  - one transport type /* str: RTP/AVP */
  - number of payloads /* unsigned int: index for easy access to payload */

each payload has:
  - one payload, /* str: 0 */
  - one encoding, /* str: PCMU */
  - one clockrate, /* str: 8000 */
  - one param /* str: NULL */



After the sdp is parsed, each stream/payload would be accessed via:
sdp_get_stream(session_index, stream_index)
sdp_get_payload(session_index, stream_index, payload_index)




Regards,
Ovidiu Sas


On 7/31/07, Bogdan-Andrei Iancu <bogdan at voice-system.ro> wrote:
> Hi Ovidiu,
>
> If this will be a collection of functions for parsing, I see no reason
> for not trying to include them in core. But for such a decision, more
> info will be needed.
>
> Regards,
> bogdan
>
> Ovidiu Sas wrote:
> > Hi all,
> >
> > I would like to know if there's interest in having a dedicated module
> > for parsing SDP.
> > This module can be re-used by nathelper and mediahelper.
> > I'm working on a qos module that is parsing the SDP, hence my interest
> > in creating a new sdpparser module or integrating the sdp parser into
> > the core.
> >
> >
> >
> > Let me know your thoughts.
> >
> > Regards,
> > Ovidiu Sas
> >
> > _______________________________________________
> > Devel mailing list
> > Devel at openser.org
> > http://openser.org/cgi-bin/mailman/listinfo/devel
> >
> >
>
>