SDL: gendynapi.py: add option to check doc formating (eg \param \returns \since)

From eea4711110af2158c7ea6a01fae01ed825f78fca Mon Sep 17 00:00:00 2001
From: Sylvain <[EMAIL REDACTED]>
Date: Sun, 12 Feb 2023 20:41:59 +0100
Subject: [PATCH] gendynapi.py: add option to check doc formating (eg \param
 \returns \since)

---
 src/dynapi/gendynapi.py | 67 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 67 insertions(+)

diff --git a/src/dynapi/gendynapi.py b/src/dynapi/gendynapi.py
index 723960ee389e..4a7d3409c275 100755
--- a/src/dynapi/gendynapi.py
+++ b/src/dynapi/gendynapi.py
@@ -320,6 +320,9 @@ def main():
     # Dump API into a json file
     full_API_json()
 
+    # Check commment formating
+    check_comment();
+
 # Dump API into a json file
 def full_API_json():
     if args.dump:
@@ -328,6 +331,69 @@ def full_API_json():
             json.dump(full_API, f, indent=4, sort_keys=True)
             print("dump API to '%s'" % filename);
 
+# Dump API into a json file
+def check_comment():
+    if args.check_comment:
+        print("check comment formating");
+
+
+        # Check \param
+        for i in full_API:
+            comment = i['comment']
+            name = i['name']
+            retval = i['retval']
+            header = i['header']
+
+            expected = len(i['parameter'])
+            if expected == 1:
+                if i['parameter'][0] == 'void':
+                    expected = 0;
+            count = comment.count("\\param")
+            if count != expected:
+                # skip SDL_stdinc.h
+                if header != 'SDL_stdinc.h':
+                    # Warning missmatch \param and function prototype
+                    print("%s: %s()  %d '\\param'' but expected %d" % (header, name, count, expected));
+
+
+        # Check \returns
+        for i in full_API:
+            comment = i['comment']
+            name = i['name']
+            retval = i['retval']
+            header = i['header']
+
+            expected = 1
+            if retval == 'void':
+                expected = 0;
+
+            count = comment.count("\\returns")
+            if count != expected:
+                # skip SDL_stdinc.h
+                if header != 'SDL_stdinc.h':
+                    # Warning missmatch \param and function prototype
+                    print("%s: %s()  %d '\\returns'' but expected %d" % (header, name, count, expected));
+
+        # Check \since
+        for i in full_API:
+            comment = i['comment']
+            name = i['name']
+            retval = i['retval']
+            header = i['header']
+
+            expected = 1
+            count = comment.count("\\since")
+            if count != expected:
+                # skip SDL_stdinc.h
+                if header != 'SDL_stdinc.h':
+                    # Warning missmatch \param and function prototype
+                    print("%s: %s()  %d '\\since'' but expected %d" % (header, name, count, expected));
+
+
+
+
+
+
 # Parse 'sdl_dynapi_procs_h' file to find existing functions
 def find_existing_procs():
     reg = re.compile('SDL_DYNAPI_PROC\([^,]*,([^,]*),.*\)')
@@ -459,6 +525,7 @@ def add_dyn_api(proc):
 
     parser = argparse.ArgumentParser()
     parser.add_argument('--dump', help='output all SDL API into a .json file', action='store_true')
+    parser.add_argument('--check-comment', help='check comment formating', action='store_true')
     parser.add_argument('--debug', help='add debug traces', action='store_true')
     args = parser.parse_args()