Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
* [ltt-dev] [LTTV PATCH] Reactivate the constructor of the filter and control dialogs
@ 2011-02-04 20:58 Yannick Brosseau
  2011-02-04 22:14 ` Mathieu Desnoyers
  0 siblings, 1 reply; 2+ messages in thread
From: Yannick Brosseau @ 2011-02-04 20:58 UTC (permalink / raw)


Both constructor were accidently removed in the warning cleanup

Signed-off-by: Yannick Brosseau <yannick.brosseau at gmail.com>
---
 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





^ permalink raw reply	[flat|nested] 2+ messages in thread

* [ltt-dev] [LTTV PATCH] Reactivate the constructor of the filter and control dialogs
  2011-02-04 20:58 [ltt-dev] [LTTV PATCH] Reactivate the constructor of the filter and control dialogs Yannick Brosseau
@ 2011-02-04 22:14 ` Mathieu Desnoyers
  0 siblings, 0 replies; 2+ messages in thread
From: Mathieu Desnoyers @ 2011-02-04 22:14 UTC (permalink / raw)


* 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 <yannick.brosseau at gmail.com>
> ---
>  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




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-02-04 22:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-04 20:58 [ltt-dev] [LTTV PATCH] Reactivate the constructor of the filter and control dialogs Yannick Brosseau
2011-02-04 22:14 ` Mathieu Desnoyers

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox