Wireshark-dev: Re: [Wireshark-dev] Wireshark no longer builds on SuSE 11.3
From: Anders Broman <[email protected]>
Date: Tue, 14 Feb 2017 12:57:19 +0000

Hi,

The code is simple enough (from glib-2.42.2 ).

**

* g_list_free_full:

* @list: a pointer to a #GList

* @free_func: the function to be called to free each element's data

*

* Convenience method, which frees all the memory used by a #GList,

* and calls @free_func on every element's data.

*

* Since: 2.28

*/

void

g_list_free_full (GList          *list,

                  GDestroyNotify  free_func)

{

  g_list_foreach (list, (GFunc) free_func, NULL);

  g_list_free (list);

}

 

To provide the function for systems not having it, Evan didn’t like the function cast though even though it’s used by glib.

 

As for dropping SuSE 11.3 it’s perhaps time…

 

Regards

Anders

 

From: [email protected] [mailto:[email protected]] On Behalf Of Roland Knall
Sent: den 14 februari 2017 13:47
To: Developer support list for Wireshark <[email protected]>
Subject: Re: [Wireshark-dev] Wireshark no longer builds on SuSE 11.3

 

I was hoping to say, let's drop SuSE 11.3, as SuSE 11.4 has glib version 2.28, which includes the function. Also ReadHat 5 drops support by the end of next month. But SLES still supports it until 2019.

 

As for the convenience method, I would not provide a compatibility header, as this would require additional checks and defines. Just hardcode the functionality in this case, although it seems to be a burden. An utility function which masks the compatibility could also be a possible solution. In extcap I came across this issue quite some time, and most of the time I ended up hardcoding the functionality.

 

regards

Roland

 

On Tue, Feb 14, 2017 at 1:28 PM, Peter Wu <[email protected]> wrote:

On Tue, Feb 14, 2017 at 12:14:24PM +0000, Anders Broman wrote:
> Hi,
> Wireshark no longer builds on SuSE 11.3
>
> home/ericsson/ewireshark/trunk/filter_files.c:104: undefined reference to `g_list_free_full'
> /home/ericsson/ewireshark/trunk/filter_files.c:105: undefined reference to `g_list_free_full'
> /home/ericsson/ewireshark/trunk/filter_files.c:106: undefined reference to `g_list_free_full'
> /home/ericsson/ewireshark/trunk/filter_files.c:107: undefined reference to `g_list_free_full'
> ui/gtk/libgtkui.a(dfilter_expr_dlg.o): In function `value_list_sel_cb':
> /home/ericsson/ewireshark/trunk/ui/gtk/dfilter_expr_dlg.c:625: undefined reference to `g_list_free_full'

This function is very useful and open-coded many times, should we
provide a compatibility header for older GLib?
--
Kind regards,
Peter Wu
https://lekensteyn.nl
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe