tree ba77d2d00f44716ad86597fb8d1bf703083dd891
parent 1d778fdce3bc4aab16dc203463d0067f2f063d16
author Jacob Erlbeck <jerlbeck@sysmocom.de> 1422017575 +0100
committer Holger Hans Peter Freyther <holger@moiji-mobile.com> 1422259806 +0100

gprs: Do not put the subscr in gprs_subscr_delete

Currently gprs_subscr_delete implicitely calls subscr_put, which
makes the code more complex than necessary (additional subscr_get) in
a few places. It also makes it more difficult to see, whether get/put
are balanced within a function. In addition, the functions are not
named consistently (gprs_subscr_delete vs.
gprs_subscr_put_and_cancel).

This commit changes the semantics of gprs_subscr_delete and
indirectly of gprs_subscr_put_and_cancel to not call subscr_put on
their argument, but to leave that for the caller to do it
explicitely.

It renames gprs_subscr_put_and_cancel to gprs_subscr_cancel to
reflect that change in the name, too.

Sponsored-by: On-Waves ehf