Modify

Opened 6 years ago

Closed 6 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 6 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

Removed by revision [8970] in version: 3.3.9.21

comment:2 Changed 6 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 6 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 6 years ago by TicketCleanup

  • Milestone 3.3.9.21 deleted

Automatic ticket cleanup.

comment:5 Changed 6 years ago by guinness

It was a mistake from the previous commit 7893. The "default" parameter should have never been added.

comment:6 Changed 6 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 6 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 6 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.

Add Comment

Modify Ticket

Action
as closed The owner will remain guinness.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.