fix: make menu handler parameters optional, this fixes Ctrl-Alt-Shift-v shortcut to show the settings window.

rcmdnk · rcmdnk · commit 595b9eb3aaca · 2025-08-11T19:32:14.000+09:00
diff --git a/lib/vim_check.ahk b/lib/vim_check.ahk
@@ -3,7 +3,7 @@
     this.Vim := Vim
   }
 
-  CheckMenu(ItemName, ItemPos, MyMenu) {
+  CheckMenu(ItemName:="", ItemPos:="", MyMenu:="") {
     ; Additional message is necessary before checking current window.
     ; Otherwise process name cannot be retrieved...?
     MsgBox("Checking current window...", "Vim Ahk")
diff --git a/lib/vim_gui.ahk b/lib/vim_gui.ahk
@@ -31,7 +31,7 @@ class VimGui{
   UpdateGui(){
   }
 
-  ShowGui(ItemName, ItemPos, MyMenu){
+  ShowGui(ItemName:="", ItemPos:="", MyMenu:=""){
     if(this.Obj == 0){
       this.Obj := Gui("", this.Title)
       this.MakeGui()
diff --git a/lib/vim_state.ahk b/lib/vim_state.ahk
@@ -43,7 +43,7 @@
     this.Vim.VimToolTip.SetRemoveToolTip(1000)
   }
 
-  FullStatus(ItemName, ItemPos, MyMenu){
+  FullStatus(ItemName:="", ItemPos:="", MyMenu:=""){
     this.CheckMode(4, , , , 1)
   }
 

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`this.Vim := Vim`
`4`	`4`	`}`
`5`	`5`
`6`		`- CheckMenu(ItemName, ItemPos, MyMenu) {`
	`6`	`+ CheckMenu(ItemName:="", ItemPos:="", MyMenu:="") {`
`7`	`7`	`; Additional message is necessary before checking current window.`
`8`	`8`	`; Otherwise process name cannot be retrieved...?`
`9`	`9`	`MsgBox("Checking current window...", "Vim Ahk")`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ class VimGui{`
`31`	`31`	`UpdateGui(){`
`32`	`32`	`}`
`33`	`33`
`34`		`- ShowGui(ItemName, ItemPos, MyMenu){`
	`34`	`+ ShowGui(ItemName:="", ItemPos:="", MyMenu:=""){`
`35`	`35`	`if(this.Obj == 0){`
`36`	`36`	`this.Obj := Gui("", this.Title)`
`37`	`37`	`this.MakeGui()`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@`
`43`	`43`	`this.Vim.VimToolTip.SetRemoveToolTip(1000)`
`44`	`44`	`}`
`45`	`45`
`46`		`- FullStatus(ItemName, ItemPos, MyMenu){`
	`46`	`+ FullStatus(ItemName:="", ItemPos:="", MyMenu:=""){`
`47`	`47`	`this.CheckMode(4, , , , 1)`
`48`	`48`	`}`
`49`	`49`