genbindings/connect: only generate connect() stubs for the fully expanded overload

cmd/genbindings/emitcabi.go

@@ -386,7 +386,7 @@ extern "C" {
 		for _, m := range c.Methods {
 			ret.WriteString(fmt.Sprintf("%s %s_%s(%s);\n", emitReturnTypeCabi(m.ReturnType), c.ClassName, m.SafeMethodName(), emitParametersCabi(m, c.ClassName+"*")))
 
-			if m.IsSignal {
+			if m.IsSignal && !m.HasHiddenParams {
 				ret.WriteString(fmt.Sprintf("%s %s_connect_%s(void* slot);\n", emitReturnTypeCabi(m.ReturnType), c.ClassName, m.SafeMethodName()))
 			}
 		}
@@ -550,7 +550,7 @@ extern "C" {
 				afterCall,
 			))
 
-			if m.IsSignal {
+			if m.IsSignal && !m.HasHiddenParams {
 				exactSignal := `static_cast<void (` + c.ClassName + `::*)(` + emitParameterTypesCpp(m) + `)>(&` + c.ClassName + `::` + nativeMethodName + `)`
 
 				ret.WriteString(

cmd/genbindings/emitgo.go

@@ -376,7 +376,7 @@ import "C"
 			`)
 
 			// Add Connect() wrappers for signal functions
-			if m.IsSignal {
+			if m.IsSignal && !m.HasHiddenParams {
 				imports["unsafe"] = struct{}{}
 				imports["runtime/cgo"] = struct{}{}
 				ret.WriteString(`func (this *` + c.ClassName + `) On` + m.SafeMethodName() + `(slot func()) {

cmd/genbindings/intermediate.go

@@ -68,6 +68,7 @@ type CppMethod struct {
 	Parameters         []CppParameter
 	IsStatic           bool
 	IsSignal           bool
+	HasHiddenParams    bool // Set to true if there is an overload with more parameters
 }
 
 func IsArgcArgv(params []CppParameter, pos int) bool {

cmd/genbindings/transformoptional.go

@@ -35,6 +35,7 @@ func astTransformOptional(parsed *CppParsedHeader) {
 					Parameters:         nil,
 					IsSignal:           m.IsSignal,
 					IsStatic:           m.IsStatic,
+					HasHiddenParams:    (x != len(m.Parameters)-1),
 				}
 				dupMethod.Parameters = append(dupMethod.Parameters, m.Parameters[0:x+1]...)
 				c.Methods = append(c.Methods, dupMethod) // TODO can we insert them next, instead of at the end?
@@ -43,6 +44,7 @@ func astTransformOptional(parsed *CppParsedHeader) {
 			// Truncate the original method's parameters to only the
 			// mandatory ones
 			m.Parameters = m.Parameters[0:optionalStart]
+			m.HasHiddenParams = true
 			c.Methods[j] = m
 		}