Opened 11 years ago
Closed 11 years ago
#2444 closed Bug (No Bug)
GUICtrlCreateMenuItem -- should the 2nd param be a default?
Reported by: | Jaber <jaberwocky6669@…> | Owned by: | guinness |
---|---|---|---|
Milestone: | Component: | AutoIt | |
Version: | 3.3.9.20 | Severity: | None |
Keywords: | Cc: |
Description
The helpfile says this:
GUICtrlCreateMenuItem ( "text", menuID = -1 [, menuentry = -1 [, menuradioitem = 0]] )
It seems like menuID should be a default parameter.
Attachments (0)
Change History (8)
comment:1 Changed 11 years ago by guinness
- Milestone set to 3.3.9.21
- Owner set to guinness
- Resolution set to Completed
- Status changed from new to closed
comment:2 Changed 11 years ago by mlipok
you talk about this:
GUICtrlCreateMenuItem ( "text" [, menuID = -1 [, menuentry = -1 [, menuradioitem = 0]]] )
If the answer is yes
then I wait too long -- it was on my list.
comment:3 Changed 11 years ago by Jpm
- Resolution Completed deleted
- Status changed from closed to reopened
Changing the doc is not enough. AutoIt code want really 2 parameters.
What is the menuID you expect to use by default?
I reopen the ticket as doc change is not enough
comment:4 Changed 11 years ago by TicketCleanup
- Milestone 3.3.9.21 deleted
Automatic ticket cleanup.
comment:5 Changed 11 years ago by guinness
It was a mistake from the previous commit 7893. The "default" parameter should have never been added.
comment:6 Changed 11 years ago by Jaber <jaberwocky6669@…>
Yes Mlipok that is what I mean and I meant make it a default in both the au source and doc.
Jpm, I guess it should stay -1 but it should be such that it can be written like this:
GUICtrlCreateMenuItem("MyMenuItemText")
comment:7 Changed 11 years ago by Jaber <jaberwocky6669@…>
But of course I now see the reason why it isn't a default parameter.
GuiCtrlCreateMenu("MyMenu") GuiCtrlCreateMenuItem("MyMenuItem1") GuiCtrlCreateMenuItem("MyMenuItem2") ; now -1 refers not to the menu but the 1st menu item. GuiCtrlCreateMenuItem("MyMenuItem3") ; now -1 refers not to the menu but the 2nd menu item.
Sorry for wasting time.
comment:8 Changed 11 years ago by Jpm
- Resolution set to No Bug
- Status changed from reopened to closed
no problem
Guidelines for posting comments:
- You cannot re-open a ticket but you may still leave a comment if you have additional information to add.
- In-depth discussions should take place on the forum.
For more information see the full version of the ticket guidelines here.
Removed by revision [8970] in version: 3.3.9.21