Alexander Malysh wrote:

Hi Paul,

+1 for the at2 part.

-1 for the reassemble part. Your patch will never work reliable because not only msisdn + refnum should be considered.
Reassemble should have triple as key: SMSC, msisdn, refnum.
IMO bb_boxc.c is the wrong place for this, bb_smscconn suites better because bb_boxc just generic connection module for external boxes and should have nothing todo with SMS magic.

agree'ing here with Alex... bb_boxc should not to "semantical SMS processing", it's the box connection layer. The bb_smscconn is the SMSC protocol abstraction layer, and MO re-assemling fits into that scope.

Stipe

-------------------------------------------------------------------
Kölner Landstrasse 419
40589 Düsseldorf, NRW, Germany

tolj.org system architecture      Kannel Software Foundation (KSF)
http://www.tolj.org/              http://www.kannel.org/

mailto:st_{at}_tolj.org           mailto:stolj_{at}_kannel.org
-------------------------------------------------------------------

Reply via email to