4:19 p.m.
From: John Ernberg <john.ernberg(a)actia.se>
v2: Bump glib requirement to version 2.32 to gain access to g_queue_free_full,
this saves using a wrapper function to clean up the queue in the stk.
John Ernberg (2):
configure: Bump glib dependecy to 2.32
stk: clean up undefined function pointer casts
configure.ac | 4 ++--
src/stk.c | 6 ++----
2 files changed, 4 insertions(+), 6 deletions(-)
--
1.9.1
4:19 p.m.
New subject: [PATCH v2 1/2] configure: Bump glib dependecy to 2.32
From: John Ernberg <john.ernberg(a)actia.se>
Needed to support g_queue_free_full
---
configure.ac | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index f6253e5..88ac6db 100644
--- a/configure.ac
+++ b/configure.ac
@@ -63,8 +63,8 @@ AC_CHECK_FUNC(signalfd, dummy=yes,
AC_CHECK_LIB(dl, dlopen, dummy=yes,
AC_MSG_ERROR(dynamic linking loader is required))
-PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.28, dummy=yes,
- AC_MSG_ERROR(GLib >= 2.28 is required))
+PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.32, dummy=yes,
+ AC_MSG_ERROR(GLib >= 2.32 is required))
AC_SUBST(GLIB_CFLAGS)
AC_SUBST(GLIB_LIBS)
--
1.9.1
4:19 p.m.
New subject: [PATCH v2 2/2] stk: clean up undefined function pointer casts
From: John Ernberg <john.ernberg(a)actia.se>
---
src/stk.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/stk.c b/src/stk.c
index 01c95b5..373f696 100644
--- a/src/stk.c
+++ b/src/stk.c
@@ -2315,8 +2315,7 @@ static gboolean handle_command_refresh(const struct stk_command
*cmd,
break;
}
- g_slist_foreach(file_list, (GFunc) g_free, NULL);
- g_slist_free(file_list);
+ g_slist_free_full(file_list, g_free);
return FALSE;
}
@@ -3163,8 +3162,7 @@ static void stk_unregister(struct ofono_atom *atom)
stk->main_menu = NULL;
}
- g_queue_foreach(stk->envelope_q, (GFunc) g_free, NULL);
- g_queue_free(stk->envelope_q);
+ g_queue_free_full(stk->envelope_q, g_free);
ofono_modem_remove_interface(modem, OFONO_STK_INTERFACE);
g_dbus_unregister_interface(conn, path, OFONO_STK_INTERFACE);
--
1.9.1
5:39 p.m.
New subject: [PATCH v2 1/2] configure: Bump glib dependecy to 2.32
On 04/26/2016 08:19 AM, John Ernberg wrote:
-PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.28, dummy=yes,
- AC_MSG_ERROR(GLib >= 2.28 is required))
+PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.32, dummy=yes,
+ AC_MSG_ERROR(GLib >= 2.32 is required))
Some time ago we had a discussion on the ConnMan mailing list doing the
same thing. IIRC, 2.32 has a larger memory footprint compared to 2.28.
That was an issue back than. Don't know if that is still relevant.
cheers,
daniel
11:11 p.m.
New subject: [PATCH v2 1/2] configure: Bump glib dependecy to 2.32
Hi Daniel,
On 04/26/2016 02:39 AM, Daniel Wagner wrote:
On 04/26/2016 08:19 AM, John Ernberg wrote:
> -PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.28, dummy=yes,
> - AC_MSG_ERROR(GLib >= 2.28 is required))
> +PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.32, dummy=yes,
> + AC_MSG_ERROR(GLib >= 2.32 is required))
Some time ago we had a discussion on the ConnMan mailing list doing the
same thing. IIRC, 2.32 has a larger memory footprint compared to 2.28.
That was an issue back than. Don't know if that is still relevant.
Yes, that is a good point. Still, I don't think it makes sense to try
and depend on a version from 2011. Newer glib versions are even worse.
Our goal is to migrate oFono away from glib and onto ell, which is the
proper solution to the footprint problem long-term.
Regards,
-Denis
11:06 p.m.
Hi John,
On 04/26/2016 01:19 AM, John Ernberg wrote:
From: John Ernberg <john.ernberg(a)actia.se>
v2: Bump glib requirement to version 2.32 to gain access to g_queue_free_full,
this saves using a wrapper function to clean up the queue in the stk.
John Ernberg (2):
configure: Bump glib dependecy to 2.32
stk: clean up undefined function pointer casts
configure.ac | 4 ++--
src/stk.c | 6 ++----
2 files changed, 4 insertions(+), 6 deletions(-)
Both applied, thanks.
Regards,
-Denis
2336
days inactive
2336
days old
5 comments
3 participants
participants (3)
-
Daniel Wagner -
Denis Kenzior -
John Ernberg