SketchUcation Premium Membership

 

 

State of Observers — 28 February 2010

Re: State of Observers — 28 February 2010

Postby AdamB » Fri Oct 15, 2010 12:25 pm

Ok, it appears to be that if you select_tool but then pop_tool, things *can* get fubar.

Mostly its benign - hence I got away with it - but with the LargeToolSet showing from Jim's Tool Organizer, the additional calls to the validation procs reveal the error. In that it is wrong to be selecting a tool then popping it, but you might hope SU would just ignore it.

The attached Ruby is a simple Tool that always performs pop_tool on keypress Escape.

It can be invoked either by calling "baddoit()" which starts it usingselect_tool, or by calling "evian()" which starts it using push_tool.
Please, register (free) to access all the attachments on the forums.
Developer of LightUp http://www.light-up.co.uk
User avatar
AdamB
LightUp
 
Posts: 744
Joined: Wed Dec 12, 2007 10:49 am
Location: Brighton, UK
Name: AdamB

Re: State of Observers — 28 February 2010

Postby thomthom » Fri Oct 15, 2010 12:32 pm

Ok, so:

select_tool + pop_tool = fubar
but
push_tool + pop_tool = OK
:?:


Thought I'd tried pop_tool with select_tool before - without pop_tool doing anything. Thought it only worked after push_tool.
Thomas Thomassen — SketchUp Monkey & Coding addict
List of my plugins and link to the CookieWare fund
User avatar
thomthom
Global Moderator
 
Posts: 17545
Joined: Tue Nov 13, 2007 12:47 pm
Location: Trondheim, Norway
Name: thomthom
Operating system: Windows
SketchUp version: 8
License type: Pro
SketchUp use: architecture
Level of SketchUp: Advanced

Re: State of Observers — 28 February 2010

Postby AdamB » Fri Oct 15, 2010 12:55 pm

thomthom wrote:Ok, so:

select_tool + pop_tool = fubar
but
push_tool + pop_tool = OK
:?:


Thought I'd tried pop_tool with select_tool before - without pop_tool doing anything. Thought it only worked after push_tool.


Not quite. select_tool + pop_tool = fubar isn't always terminal. Just apparently if there are really large toolbars around hence its never caused a problem for me in the past.

Thomthom, can you get Jims' Organiser thing and see if you can repro this.
Developer of LightUp http://www.light-up.co.uk
User avatar
AdamB
LightUp
 
Posts: 744
Joined: Wed Dec 12, 2007 10:49 am
Location: Brighton, UK
Name: AdamB

Re: State of Observers — 28 February 2010

Postby thomthom » Fri Oct 15, 2010 12:59 pm

Size of toolbars matter?
Or is it toolbars with validation procs?

I have Jim's plugin installed on my Sy7 installation. Is it a particular pre-set toolbar that should be tested against?
Thomas Thomassen — SketchUp Monkey & Coding addict
List of my plugins and link to the CookieWare fund
User avatar
thomthom
Global Moderator
 
Posts: 17545
Joined: Tue Nov 13, 2007 12:47 pm
Location: Trondheim, Norway
Name: thomthom
Operating system: Windows
SketchUp version: 8
License type: Pro
SketchUp use: architecture
Level of SketchUp: Advanced

Re: State of Observers — 28 February 2010

Postby Dan Rathbun » Fri Mar 18, 2011 5:15 am

AdamB wrote: After I'm done with a Tool, I always done .pop_tool rather than .select_tool(nil) because it seems less presumptuous. ie restore what the user was doing before rather than cancel their previous selection.

However, Gaieus found out a problem/conflict with Jim Toolbar Organizer - long story short, I've switched to doing select_tool(nil) to avoid some weird race-condition with menu validation procs.

This issue is caused by the tool stack having no tool_id (or more precisely Tools.active_tool_id == 0 ).

API Doc wrote:NOTE: Calling active_tool_name on an empty Tools collection might cause a crash. Before calling this method, it is important to check whether or not the Tools collection is empty. The Tools collection is empty if the method .active_tool_id returns zero.


So always check if tools.active_tool_id==0 before using .pop_tool() or .active_tool_name()
User avatar
Dan Rathbun
Top SketchUcator
 
Posts: 4069
Joined: Tue Oct 06, 2009 3:06 am
Location: Florida, USA
Name: Dan Rathbun
Operating system: Windows
SketchUp version: 8
License type: Pro
SketchUp use: education
Level of SketchUp: Advanced

Re: State of Observers — 28 February 2010

Postby Dan Rathbun » Fri Mar 18, 2011 5:26 am


onQuit() callback

I have noticed that on SU 8.0M1 (at least,) that Sketchup does not wait until the callback returns, before it closes all it's child and owned popups, and it's own app window and shuts down.

You can still have code in an onQuit() callback running after all Sketchup's windows have closed. (You can try it with a messagebox.)

I was trying to save size and position of some windows, but Sketchup closes all it's windows before my onQuit() callback code can use Win32API calls to grab the window sizes. I even 'leaned' the code out as much as possible, but still to no avail, ... my code writes empty size arrays into the registry.

My next choice will be to try and use Ruby's built-in define_finalizer for one of my objects, perhaps the plugin module, or the observer object, .. and see if Sketchup will wait until the finalizer block returns.

Did anyone notice this behaviour in SU 7.x ?
User avatar
Dan Rathbun
Top SketchUcator
 
Posts: 4069
Joined: Tue Oct 06, 2009 3:06 am
Location: Florida, USA
Name: Dan Rathbun
Operating system: Windows
SketchUp version: 8
License type: Pro
SketchUp use: education
Level of SketchUp: Advanced

Re: State of Observers — 28 February 2010

Postby Dan Rathbun » Mon Mar 21, 2011 3:35 pm

Quick Question ...

Seems I remember that AppObserver::onNewModel does not get called when Sketchup starts up.

Is this generally true.. or specific to PC or Mac ?

I see the API says that a command line skp file will not fire the onOpenModel() callback.
User avatar
Dan Rathbun
Top SketchUcator
 
Posts: 4069
Joined: Tue Oct 06, 2009 3:06 am
Location: Florida, USA
Name: Dan Rathbun
Operating system: Windows
SketchUp version: 8
License type: Pro
SketchUp use: education
Level of SketchUp: Advanced

Re: State of Observers — 28 February 2010

Postby thomthom » Mon Mar 21, 2011 3:39 pm

Dan Rathbun wrote:Seems I remember that AppObserver::onNewModel does not get called when Sketchup starts up.

Is this generally true.. or specific to PC or Mac ?

Aye.
Thomas Thomassen — SketchUp Monkey & Coding addict
List of my plugins and link to the CookieWare fund
User avatar
thomthom
Global Moderator
 
Posts: 17545
Joined: Tue Nov 13, 2007 12:47 pm
Location: Trondheim, Norway
Name: thomthom
Operating system: Windows
SketchUp version: 8
License type: Pro
SketchUp use: architecture
Level of SketchUp: Advanced

SketchUcation One-Liner Adverts

by Ad Machine » 5 minutes ago

Need Authorised SketchUp Training by experts in various disciplines? Check out our Training details.

Ad Machine
Robot
 
Posts: 2012

Previous

Return to Developers' Forum

Who is online

Users browsing this forum: Flippie123 and 5 guests