[PATCH 2/9] libmsc: Add USSD MAP constant in gsup_message_type

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Neels Hofmeyr nhofmeyr at sysmocom.de
Tue Apr 26 11:39:18 UTC 2016


Just minor remarks...

IMHO it would be good to mention in the commit log the file/function in
which the constant will be used, if not adding the constant along with its
first use.

Why are you adding the GPRS_GSUP_MSGT_USSD_MAP out of sequence?
So far the constants are sorted by bit value, rather comply to that.

~Neels

On Fri, Apr 22, 2016 at 02:41:31PM +0200, Sergey Kostanbaev wrote:
> From: Sergey Kostanbaev <Sergey.Kostanbaev at fairwaves.co>
> 
> ---
>  openbsc/include/openbsc/gprs_gsup_messages.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/openbsc/include/openbsc/gprs_gsup_messages.h b/openbsc/include/openbsc/gprs_gsup_messages.h
> index 8cbc809..fb9c66e 100644
> --- a/openbsc/include/openbsc/gprs_gsup_messages.h
> +++ b/openbsc/include/openbsc/gprs_gsup_messages.h
> @@ -53,6 +53,8 @@ enum gprs_gsup_iei {
>  };
>  
>  enum gprs_gsup_message_type {
> +	GPRS_GSUP_MSGT_USSD_MAP                 = 0b01111111,
> +
>  	GPRS_GSUP_MSGT_UPDATE_LOCATION_REQUEST	= 0b00000100,
>  	GPRS_GSUP_MSGT_UPDATE_LOCATION_ERROR	= 0b00000101,
>  	GPRS_GSUP_MSGT_UPDATE_LOCATION_RESULT	= 0b00000110,
> -- 
> 1.9.1
> 

-- 
- Neels Hofmeyr <nhofmeyr at sysmocom.de>          http://www.sysmocom.de/
=======================================================================
* sysmocom - systems for mobile communications GmbH
* Alt-Moabit 93
* 10559 Berlin, Germany
* Sitz / Registered office: Berlin, HRB 134158 B
* Geschäftsführer / Managing Directors: Holger Freyther, Harald Welte
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.osmocom.org/pipermail/openbsc/attachments/20160426/92bbe106/attachment.bin>


More information about the OpenBSC mailing list