Hi Ivan!

2013/8/7 <[email protected]>

> Hi all!
>
> I've changed some common functionality... in my branch
> May be it will help somebody.
>

For those who doesn't know it, Ivan's repository is this one:
https://github.com/blackwarthog/synfig


> Functionality that applies default layer parameters was extracted from
> method CanvasInterface::add_layer_to() into separate method
> CanvasInterface::apply_layer_param_defaults()
>

Yeah, it is always good to modularize common tasks


> Action::System::perform_action() which uses for performing action not
> supports recursive calls. So we cannot call
> Action::System::perform_action() again while performing action. And cannot
> call CanvasInterface::add_layer_to() too.
>

That's right, perform_action cannot be called while perform_action is
running. I don't find easily a case where it is useful. Can you show us
when you needed to call perform_action inside the perform_action function?
I just can think on fractals or recursive drawing for that kind of cases. I
think that recursive actions can be done using the Super action including
the action itself inside the Super action (obviously with a condition to
get out of the recursive)

In any case good work!

>
>
> --
> Ivan
>
>
>
> ------------------------------------------------------------------------------
> Get 100% visibility into Java/.NET code with AppDynamics Lite!
> It's a free troubleshooting tool designed for production.
> Get down to code-level detail for bottlenecks, with <2% overhead.
> Download for free and get started troubleshooting in minutes.
> http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
> _______________________________________________
> Synfig-devl mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/synfig-devl
>



-- 
Carlos
http://synfig.org
------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead. 
Download for free and get started troubleshooting in minutes. 
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
Synfig-devl mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/synfig-devl

Reply via email to