* [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