Re: Temporary tools

[Valentin Ungureanu]

Bernhard Herzog wrote:
> I've commited the patch now together with fixes for these two issues.
> I've solved 2. a bit differently than per-editor-tools2.diff, though.
> See the code.

AFAICT everything works fine.
I see that SketchCanvas.SetTool still exists. Is it still needed or it
just slipped through?

> > The most notable differences where that context.tool was a property (and
> > assigning to context.tool called context.SetTool)
> 
> Well, Context is a classic class, and for those properties don'r work
> properly.  __get__ is called, but not __set__ IIRC.

Yes, making Context a new-style class was part of making tool a
property, I just didn't mention it explicitely.  It doesn't matter
anyway, all works fine now.


I updated the patch for temporary tools as suggested, any comments are
welcome:
https://savannah.nongnu.org/patch/index.php?func=detailitem&item_id=4516