Parse function pointers more correctly in the PyAPI_DATA headers check

encukou · encukou · commit de489c07bcfe · 2021-04-23T15:06:58.000+02:00
diff --git a/Tools/scripts/stable_abi.py b/Tools/scripts/stable_abi.py
@@ -459,7 +459,7 @@ def gcc_get_limited_api_definitions(headers):
         re.findall(r"__EXPORT_DATA\((.*?)\)", preprocesor_output)
     )
     stable_data = set(
-        re.findall(r"__PyAPI_DATA\(.*?\)\s*\(?(.*?)\)?\s*;", preprocesor_output)
+        re.findall(r"__PyAPI_DATA\(.*?\)[\s\*\(]*([^);]*)\)?.*;", preprocesor_output)
     )
     return stable_data | stable_exported_data | stable_functions
 

Original file line number	Diff line number	Diff line change
`@@ -459,7 +459,7 @@ def gcc_get_limited_api_definitions(headers):`
`459`	`459`	`re.findall(r"__EXPORT_DATA\((.*?)\)", preprocesor_output)`
`460`	`460`	`)`
`461`	`461`	`stable_data = set(`
`462`		`- re.findall(r"__PyAPI_DATA\(.?\)\s\(?(.?)\)?\s;", preprocesor_output)`
	`462`	`+ re.findall(r"__PyAPI_DATA\(.?\)[\s\\(]([^);])\)?.*;", preprocesor_output)`
`463`	`463`	`)`
`464`	`464`	`return stable_data \| stable_exported_data \| stable_functions`
`465`	`465`