[PATCH 1/2] Refactor the Uplink RLC header parsing function
laforge at gnumonks.org
Thu Mar 24 22:45:06 UTC 2016
On Tue, Mar 22, 2016 at 06:42:30PM +0530, Bhargava Abhyankar wrote:
> Parsing the uplink data header for GPRS and EGPRS header type 3
> is handled in separate functions.
> This patch will enhance modularity of the code.
I'm not an expert on OsmoPCU, but this patch looks pretty
straight-forward and undisputable. Moving away from large switch
statements towards separate functions/methods is generally appreciated
a lot, as is increasing modularity.
- Harald Welte <laforge at gnumonks.org> http://laforge.gnumonks.org/
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)
More information about the osmocom-net-gprs