diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-11 13:37:00 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-30 19:09:52 +0100 |
| commit | afc314e5967400e400c45633a5ce088351c60938 (patch) | |
| tree | 0aec16db73eb958d4af02cba958d71270dc818b7 /src/mm-port-serial-qcdm.c | |
| parent | 8d3d1183b9448ae9f4b87dda66aaf3314563b427 (diff) | |
port-serial-qcdm: fix warnings with -Wsign-compare
mm-port-serial-qcdm.c: In function ‘find_qcdm_start’:
mm-port-serial-qcdm.c:51:19: error: comparison of integer expressions of different signedness: ‘int’ and ‘guint’ {aka ‘unsigned int’} [-Werror=sign-compare]
51 | for (i = 0; i < response->len; i++) {
| ^
Diffstat (limited to 'src/mm-port-serial-qcdm.c')
| -rw-r--r-- | src/mm-port-serial-qcdm.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/mm-port-serial-qcdm.c b/src/mm-port-serial-qcdm.c index ec1833f5..d343df91 100644 --- a/src/mm-port-serial-qcdm.c +++ b/src/mm-port-serial-qcdm.c @@ -40,7 +40,8 @@ struct _MMPortSerialQcdmPrivate { static gboolean find_qcdm_start (GByteArray *response, gsize *start) { - int i, last = -1; + guint i; + gint last = -1; /* Look for 3 bytes and a QCDM frame marker, ie enough data for a valid * frame. There will usually be three cases here; (1) a QCDM frame @@ -50,7 +51,7 @@ find_qcdm_start (GByteArray *response, gsize *start) */ for (i = 0; i < response->len; i++) { if (response->data[i] == 0x7E) { - if (i > last + 3) { + if (i > (guint)(last + 3)) { /* Got a full QCDM frame; 3 non-0x7E bytes and a terminator */ if (start) *start = last + 1; |