Re: Opwnsmppbox patch
Aarno Syvänen <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi Alex a) I can add code for other DB tzpes, but someone lese must test them. I do not have Oracle ;) b) Will fix the style c) Opensmppbox uses 8 char id. V3.3 issue, i think. d) I do not like flags, generally. But I will, of course, fix the style. e) Opensmppbox uses the patch. Do you think that this should be part of opensmppbox instead ? Aarno On Dec 19, 2011, at 8:11 PM, Alexander Malysh wrote: > Hi Aarno, > > I don't like gw dlr part at all. It's: > a) not complete, not all all DB types implemented > b) not C style at some places (variables defined within function) > c) why use only 8 chars when using smpp_id? > d) why not add flag to dlr_find which decide to always add DLR to storage? > e) I don't see use cases > > Thanks, > Alex > > Am 19.12.2011 um 17:15 schrieb Aarno Syvänen: > >> Hi List, >> >> here are diffs again >> >> to Kannel, for documentation >> <doc.diff> >> >> for gw >> >> <gw.diff> >> >> for gwlib >> >> <gwlib.diff> >> >> for opensmppbox, for documentation >> >> <doc.diff> >> >> for code >> <box.diff> >> >> Aarno >> > >