From mboxrd@z Thu Jan 1 00:00:00 1970 From: compudj@krystal.dyndns.org (Mathieu Desnoyers) Date: Fri, 4 Feb 2011 17:14:18 -0500 Subject: [ltt-dev] [LTTV PATCH] Reactivate the constructor of the filter and control dialogs In-Reply-To: <1296853116-17298-1-git-send-email-yannick.brosseau@gmail.com> References: <1296853116-17298-1-git-send-email-yannick.brosseau@gmail.com> Message-ID: * Yannick Brosseau (yannick.brosseau at gmail.com) wrote: > Both constructor were accidently removed in the warning cleanup > Merged, thanks! Mathieu > Signed-off-by: Yannick Brosseau > --- > lttv/modules/gui/filter/filter.c | 5 ++++- > lttv/modules/gui/tracecontrol/tracecontrol.c | 6 +++++- > 2 files changed, 9 insertions(+), 2 deletions(-) > > diff --git a/lttv/modules/gui/filter/filter.c b/lttv/modules/gui/filter/filter.c > index 2357465..a69685f 100644 > --- a/lttv/modules/gui/filter/filter.c > +++ b/lttv/modules/gui/filter/filter.c > @@ -462,7 +462,10 @@ gui_filter_destructor(FilterViewerData *fvd) > GtkWidget * > h_guifilter(LttvPlugin *plugin) > { > - return NULL; > + gui_filter(plugin); > + > + /* TODO ybrosseau 2011-02-04: We should probably return a widget here */ > + return NULL; > } > > /** > diff --git a/lttv/modules/gui/tracecontrol/tracecontrol.c b/lttv/modules/gui/tracecontrol/tracecontrol.c > index a02301b..56165d9 100644 > --- a/lttv/modules/gui/tracecontrol/tracecontrol.c > +++ b/lttv/modules/gui/tracecontrol/tracecontrol.c > @@ -1242,7 +1242,11 @@ void disarm_clicked (GtkButton *button, gpointer user_data) > GtkWidget * > h_guicontrol(LttvPlugin *plugin) > { > - return NULL; > + LttvPluginTab *ptab = LTTV_PLUGIN_TAB(plugin); > + gui_control(ptab) ; > + > + /* TODO ybrosseau 2011-02-04: We should probably return a widget here */ > + return NULL; > } > > /** > -- > 1.7.2.3 > > > _______________________________________________ > ltt-dev mailing list > ltt-dev at lists.casi.polymtl.ca > http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev > -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com