From 7a0bac9dfc8b5df249c18c74c40821ac9eadcf5d Mon Sep 17 00:00:00 2001
From: ceresgalax <ceresgalax@gmail.com>
Date: Sun, 11 Feb 2024 15:47:51 -0800
Subject: [PATCH] Make Decl.DescribedTemplate property nullable.

Obviously, many Decl cursors will not describe a template. Make this property null to reflect that an avoid an exception by TranslationUnit.GetOrCreate<TemplateDecl>().

Additionally, DescribedTemplate will be null for partial template specializations. Added a doc comment to the property as this is less obvious.
---
 sources/ClangSharp/Cursors/Decls/Decl.cs | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/sources/ClangSharp/Cursors/Decls/Decl.cs b/sources/ClangSharp/Cursors/Decls/Decl.cs
index e509e4a5..7274bae8 100644
--- a/sources/ClangSharp/Cursors/Decls/Decl.cs
+++ b/sources/ClangSharp/Cursors/Decls/Decl.cs
@@ -15,7 +15,7 @@ public class Decl : Cursor
     private readonly Lazy<Stmt?> _body;
     private readonly Lazy<Decl> _canonicalDecl;
     private readonly Lazy<IReadOnlyList<Decl>> _decls;
-    private readonly Lazy<TemplateDecl> _describedTemplate;
+    private readonly Lazy<TemplateDecl?> _describedTemplate;
     private readonly Lazy<Decl> _mostRecentDecl;
     private readonly Lazy<Decl> _nextDeclInContext;
     private readonly Lazy<Decl> _nonClosureContext;
@@ -62,7 +62,11 @@ private protected Decl(CXCursor handle, CXCursorKind expectedCursorKind, CX_Decl
             return decls;
         });
 ;
-        _describedTemplate = new Lazy<TemplateDecl>(() => TranslationUnit.GetOrCreate<TemplateDecl>(Handle.DescribedTemplate));
+        _describedTemplate = new Lazy<TemplateDecl?>(() =>
+        {
+            CXCursor describedTemplate = Handle.DescribedTemplate;
+            return describedTemplate.IsNull ? null : TranslationUnit.GetOrCreate<TemplateDecl>(describedTemplate);
+        });
         _mostRecentDecl = new Lazy<Decl>(() => TranslationUnit.GetOrCreate<Decl>(Handle.MostRecentDecl));
         _nextDeclInContext = new Lazy<Decl>(() => TranslationUnit.GetOrCreate<Decl>(Handle.NextDeclInContext));
         _nonClosureContext = new Lazy<Decl>(() => TranslationUnit.GetOrCreate<Decl>(Handle.NonClosureContext));
@@ -90,7 +94,11 @@ private protected Decl(CXCursor handle, CXCursorKind expectedCursorKind, CX_Decl
 
     public IReadOnlyList<Decl> Decls => _decls.Value;
 
-    public TemplateDecl DescribedTemplate => _describedTemplate.Value;
+    /// <summary>
+    /// Per clang documentation: This returns null for partial specializations, because they are not modeled as TemplateDecls.
+    /// Use DescribedTemplateParams to handle those cases.
+    /// </summary>
+    public TemplateDecl? DescribedTemplate => _describedTemplate.Value;
 
     public bool HasAttrs => Handle.HasAttrs;