If I understand correctly your fix solution, you're setting up a pair of
callbacks to update the keyframe list by the two widgets but they avoid to
update it when the emitter of the callback is the widget itself (so you
avoid a endless loop).
Using void* is the same than using gpointer, since the last if s typedef of
the first.
I only have one idea that is to use a GtkWidget* and cast to the emitter
type (Widget_Keyframe_List or KeyframeTree) on each case. I don't think
that it would add any extra functionality and if with void* works, then
leave it like that. ^__^
Maybe just add one comment about that on the code to give a hint to whom
would look at it in the future.
Cheers!
2014/1/6 <[email protected]>
> hello,
>
> In order to solve infinite loop between send/receive signal in the
> "keyframe synchro" patch, i have already
> added "send_signal" boolean test.
>
> https://github.com/d-j-a-y/synfig/commit/f3fcca7863c580bb1752dd2e8fafecfb010fba63
>
> This commit, i have added a emitter/recipient test based on "this" pointer.
>
> https://github.com/d-j-a-y/synfig/commit/5183c98a56f96cab0bc173d24664fb00bf62bda6
>
> Has synfigapp/canvasinterface.h do not include nor gtkmm , glib, i have
> used c style cast to (void*).
> And i don't feel that a very nice (portable?) cpp way to do.
>
> Can you confirm it's portable (sys/arch) enough ?
> Does i should add glib support in synfigapp/canvasinterface.h to have a
> nice gpointer instead of void* ?
> Any other suggestion ?
>
> seevoid(ya!
>
>
>
> ------------------------------------------------------------------------------
> Rapidly troubleshoot problems before they affect your business. Most IT
> organizations don't have a clear picture of how application performance
> affects their revenue. With AppDynamics, you get 100% visibility into your
> Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics
> Pro!
> http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
> _______________________________________________
> Synfig-devl mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/synfig-devl
>
>
--
Carlos
http://synfig.org
------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT
organizations don't have a clear picture of how application performance
affects their revenue. With AppDynamics, you get 100% visibility into your
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
_______________________________________________
Synfig-devl mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/synfig-devl
