Review at  https://gerrit.osmocom.org/2964

gsm0411_utils.c: Fix compilation warnings

gsm0411_utils.c:102:2: warning: #warning find a portable way to obtain timezone 
offset [-Wcpp]
 #warning find a portable way to obtain timezone offset

gsm0411_utils.c: In function 'gsm338_get_sms_alphabet':
gsm0411_utils.c:260:4: warning: large integer implicitly truncated to unsigned 
type [-Woverflow]
    return 0xffffffff;

Change-Id: I1d6cb31f38721f79e2cf93f9b8e4776f3720aa07
---
M src/gsm/gsm0411_utils.c
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/64/2964/1

diff --git a/src/gsm/gsm0411_utils.c b/src/gsm/gsm0411_utils.c
index 543775c..089df17 100644
--- a/src/gsm/gsm0411_utils.c
+++ b/src/gsm/gsm0411_utils.c
@@ -99,7 +99,7 @@
 #ifdef HAVE_TM_GMTOFF_IN_TM
        *scts++ = gsm411_bcdify(tm->tm_gmtoff/(60*15));
 #else
-#warning find a portable way to obtain timezone offset
+#pragma message ("find a portable way to obtain timezone offset")
        *scts++ = 0;
 #endif
 }
@@ -257,7 +257,7 @@
                if (cgbits & 2) {
                        LOGP(DLSMS, LOGL_NOTICE,
                             "Compressed SMS not supported yet\n");
-                       return 0xffffffff;
+                       return -1;
                }
 
                switch ((dcs >> 2)&0x03) {

-- 
To view, visit https://gerrit.osmocom.org/2964
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d6cb31f38721f79e2cf93f9b8e4776f3720aa07
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <[email protected]>

Reply via email to