>>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). 

Yep exactly 





>>Using void* is the same than using gpointer, since the last if s typedef of 
>>the first. 
Yep, that why i thought i was cleaner. 
Anyway, the void pointer size will be fixed at building time in both case. 

>>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. 
That was my first idea, but same issue that gpointer, canvasinterface.h would 
need gtkmm*.h included. 

>>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. 
Yep. 

Cheers! 





2014/1/6 < d.j....@free.fr > 




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 
Synfig-devl@lists.sourceforge.net 
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
Synfig-devl@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/synfig-devl

Reply via email to