Some structures should be passed to Py_BuildValue by reference, not by value,

jackjansen · jackjansen · commit 89dbd972799c · 2002-03-18T15:24:22.000Z
notably FSSpec and FSRef objects. First half of fix for #531291.

2.2.1 candidate.
diff --git a/Tools/bgen/bgen/bgenType.py b/Tools/bgen/bgen/bgenType.py
@@ -233,6 +233,14 @@ def passInput(self, name):
 
 	def mkvalueArgs(self, name):
 		return "%s, %s" % (self.new, name)
+		
+class OpaqueByValueStructType(OpaqueByValueType):
+	"""Similar to OpaqueByValueType, but we also pass this to mkvalue by
+	address, in stead of by value.
+	"""
+
+	def mkvalueArgs(self, name):
+		return "%s, &%s" % (self.new, name)
 
 
 class OpaqueArrayType(OpaqueByValueType):
diff --git a/Tools/bgen/bgen/macsupport.py b/Tools/bgen/bgen/macsupport.py
@@ -46,9 +46,9 @@
 
 # File System Specifications
 FSSpec_ptr = OpaqueType("FSSpec", "PyMac_BuildFSSpec", "PyMac_GetFSSpec")
-FSSpec = OpaqueByValueType("FSSpec", "PyMac_BuildFSSpec", "PyMac_GetFSSpec")
+FSSpec = OpaqueByValueStructType("FSSpec", "PyMac_BuildFSSpec", "PyMac_GetFSSpec")
 FSRef_ptr = OpaqueType("FSRef", "PyMac_BuildFSRef", "PyMac_GetFSRef")
-FSRef = OpaqueByValueType("FSRef", "PyMac_BuildFSRef", "PyMac_GetFSRef")
+FSRef = OpaqueByValueStructType("FSRef", "PyMac_BuildFSRef", "PyMac_GetFSRef")
 
 # OSType and ResType: 4-byte character strings
 def OSTypeType(typename):
