<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 25, 2015 at 3:31 PM, Holger Freyther <span dir="ltr"><<a href="mailto:holger@freyther.de" target="_blank">holger@freyther.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class=""><br>
> On 25 Nov 2015, at 13:22, Holger Freyther <<a href="mailto:holger@freyther.de">holger@freyther.de</a>> wrote:<br>
><br>
><br>
><br>
> Have you considered using SMPP for USSD? At first this looks more natural<br>
> than a custom protocol like SUP.<br>
<br>
<br>
</span>The other part I notice when going through the patches are the following:<br>
<br>
* Fixes, features without a unit test is very difficult to sell<br></blockquote><div>I agree that having a unit test is a good sign. But I'm not really familiar with the project rules.<br><br></div><div>I can make unit tests for packet parsing/composing (SS and MAP-like level). But I have no idea how to make a unit test for e.g. USSD transaction based on gsm_trans or more complex things. I'd be glad to hear your thoughts.<br><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
* SUP has a protocol description. If the protocol is changed the description should<br>
be updated.<br></blockquote><div>I think I don't get this. Which protocol description are you talking about? <br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
* If gprs sup client is used in non GPRS context we either split the protocol up or<br>
at least rename and move these files around.<br></blockquote><div> </div><div>I intentionally left this part as is to get a feedback how it's possible solve. So you suggest to separate abstract socket operations away from GPRS, right?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
If you have questions of any of the above points I am happy to answer them.<br>
<span class=""><font color="#888888"><br>
holger</font></span></blockquote></div><br></div></div>