[gobject-introspection/tintou/closure-destroy-fix: 2/4] Fix inconsistencies when handling closure and destroy annotations

[Christoph Reiter <creiter src gnome org>]

commit c0c60dc2ef6210665f1609023ff1a44f8eae2b8c
Author: Corentin Noël <corentin elementary io>
Date:   Wed Jul 31 16:11:18 2019 +0200

    Fix inconsistencies when handling closure and destroy annotations

 giscanner/maintransformer.py | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)
---
diff --git a/giscanner/maintransformer.py b/giscanner/maintransformer.py
index 9468751d..e93b40ad 100644
--- a/giscanner/maintransformer.py
+++ b/giscanner/maintransformer.py
@@ -774,22 +774,26 @@ class MainTransformer(object):
 
             destroy_annotation = annotations.get(ANN_DESTROY)
             if destroy_annotation:
-                param.destroy_name = self._get_validate_parameter_name(parent,
-                                                                       destroy_annotation[0],
-                                                                       param)
-                if param.destroy_name is not None:
+                destroy_param = parent.get_parameter(destroy_annotation[0])
+                if destroy_param:
+                    destroy_param.destroy_name = param.argname
                     param.scope = ast.PARAM_SCOPE_NOTIFIED
-                    destroy_param = parent.get_parameter(param.destroy_name)
                     # This is technically bogus; we're setting the scope on the destroy
                     # itself.  But this helps avoid tripping a warning from finaltransformer,
                     # since we don't have a way right now to flag this callback a destroy.
                     destroy_param.scope = ast.PARAM_SCOPE_NOTIFIED
+                else:
+                    message.warn('%s: unable to find parameter %s' % (param.argname, destroy_annotation[0],),
+                                 tag.position)
 
             closure_annotation = annotations.get(ANN_CLOSURE)
             if closure_annotation and len(closure_annotation) == 1:
-                param.closure_name = self._get_validate_parameter_name(parent,
-                                                                   closure_annotation[0],
-                                                                   param)
+                closure_param = parent.get_parameter(closure_annotation[0])
+                if closure_param:
+                    closure_param.closure_name = param.argname
+                else:
+                    message.warn('%s: unable to find parameter %s' % (param.argname, closure_annotation[0],),
+                                 tag.position)
         elif isinstance(parent, ast.Callback):
             if ANN_CLOSURE in annotations:
                 # For callbacks, (closure) appears without an