ChangeWorkbenchSelection()

5 posts / 0 new

Last post

Sun, 2015-06-14 22:03

mritter0

Offline

Last seen: 1 year 10 months ago

Joined: 2014-04-21 21:15

ChangeWorkbenchSelection()

I can get the example code on the AutoDocs wiki to work, but it only gives me the first icon. How do I loop through to get all the icon names in the path? There is no way the example code could compare all the names to "Prefs" the way it is shown.

Tue, 2015-06-16 04:05

broadblues

Offline

Last seen: 4 years 2 months ago

Joined: 2012-05-02 21:48

The example code looks wrong

The example code looks wrong to me.

It should not be returning BOOL but one of three values.

try something like:

	ULONG
	SelectPrefs(struct Hook *hook, APTR reserved,
	            struct IconSelectMsg *ism)
	{
 
	    ULONG result = ISMACTION_Ignore;
 
 
	    // If the name matches, select it. Otherwise, leave its
	    // select state alone.
	    if(Stricmp(ism->ism_Name,"Prefs") == 0)
	        result = ISMACTION_Select;
 
	    return(result);
	}

[edit]

Raised a BZ against the OS component.

Wed, 2015-06-17 00:17

mritter0

Offline

Last seen: 1 year 10 months ago

Joined: 2014-04-21 21:15

I saw that too and am using

I saw that too and am using the ignore tag. But I still don't know how to loop through the list of icons. The example shows only comparing the name once.

for(node=GetHead(list); node; node->GetSucc(list)) // or whatever the syntax is
{
....compare names
....return(found it)
}
return(didn't find it)

Wed, 2015-06-17 01:23

(Reply to #3) #4

AND

Offline

Last seen: 6 years 10 months ago

Joined: 2011-01-20 12:22

Your hook is called for each

Your hook is called for each icon inside the drawer, given as the first argument in ChangeWorkbenchSelection().
So, the loop is done by workbench and not by the application.

You can stop the loop, by returning ISMACTION_Stop.

So, lets say the drawer LoopMe has three icons Icon1, Icon2 and Icon3. Then SelectIcon is called three times:

select_hook.h_Entry = (HOOKFUNC)SelectIcon;
ChangeWorkbenchSelection("LoopMe", &select_hook, NULL);
 
SelectIcon() -> IconSelectMsg->ism_Name = Icon1
SelectIcon() -> IconSelectMsg->ism_Name = Icon2
SelectIcon() -> IconSelectMsg->ism_Name = Icon3

Wed, 2015-06-17 02:21

mritter0

Offline

Last seen: 1 year 10 months ago

Joined: 2014-04-21 21:15

@AND: That is what I thought

@AND: That is what I thought it was supposed to do (and it is). I was totally not catching the names properly. My bad. It works as described. I am doing something totally different than the example.

Search form

User login