SimpleCommand now without IActiveAware interface?

Jul 16, 2010 at 12:28 PM

Hallo Sacha...

in your last built 48484 the simple command does not include the IActiveAware interfac.

This isactiv function was very usefull.

Is it a problem to include this interface again or is it responsible for the memory leak?

 

Best regards Avidin

Coordinator
Jul 19, 2010 at 12:48 PM

No I could add it again, I just didn't use it myself, if you want it I will look to add it back in. 

Coordinator
Jul 19, 2010 at 7:49 PM

Avidin

 

After much thought I have decided to omit the IActiveAware from SimpleCommand<T1,T2> as I did have a memory leak in there to do with CommandCompleted event (which I use in a TargettedTrigger), so I do not want to introduce a problem. But if you want that you can just inherit from SimpleCommand<T1,T2> and also add that to your own codebase.

 

Sorry about this decision, but that is the one I am making.