From 49988f22b9e0a7dc2977fc1e8784bb6aa63e368f Mon Sep 17 00:00:00 2001
From: jeff <jeff@rvhome.de>
Date: Sun, 16 May 2021 17:01:44 +0200
Subject: [PATCH] use py_scadparser

---
 solid/objects.py     | 11 +---------
 solid/solidpython.py | 49 ++++++--------------------------------------
 2 files changed, 7 insertions(+), 53 deletions(-)

diff --git a/solid/objects.py b/solid/objects.py
index b1409e6a..2307e804 100644
--- a/solid/objects.py
+++ b/solid/objects.py
@@ -853,16 +853,7 @@ def use(scad_file_path: PathStr, use_not_include: bool = True, dest_namespace_di
 
     scad_file_path = _find_library(scad_file_path) 
 
-    contents = None
-    try:
-        contents = scad_file_path.read_text()
-    except Exception as e:
-        raise Exception(f"Failed to import SCAD module '{scad_file_path}' with error: {e} ")
-
-    # Once we have a list of all callables and arguments, dynamically
-    # add OpenSCADObject subclasses for all callables to the calling module's
-    # namespace.
-    symbols_dicts = parse_scad_callables(contents)
+    symbols_dicts = parse_scad_callables(scad_file_path)
 
     for sd in symbols_dicts:
         class_str = new_openscad_class_str(sd['name'], sd['args'], sd['kwargs'],
diff --git a/solid/solidpython.py b/solid/solidpython.py
index f1a1b8cd..9562e8c4 100755
--- a/solid/solidpython.py
+++ b/solid/solidpython.py
@@ -611,51 +611,14 @@ def sp_code_in_scad_comment(calling_file: PathStr) -> str:
 # ===========
 # = Parsing =
 # ===========
+def parse_scad_callables(filename: str) -> List[dict]:
+    from .py_scadparser import scad_parser
 
-def parse_scad_callables(scad_code_str: str) -> List[dict]:
-    callables = []
+    _, _, modules, functions, _ = scad_parser.parseFile(filename)
 
-    # Note that this isn't comprehensive; tuples or nested data structures in
-    # a module definition will defeat it.
-
-    # Current implementation would throw an error if you tried to call a(x, y)
-    # since Python would expect a(x);  OpenSCAD itself ignores extra arguments,
-    # but that's not really preferable behavior
-
-    # TODO:  write a pyparsing grammar for OpenSCAD, or, even better, use the yacc parse grammar
-    # used by the language itself.  -ETJ 06 Feb 2011
-
-    # FIXME: OpenSCAD use/import includes top level variables. We should parse 
-    # those out (e.g. x = someValue;) as well -ETJ 21 May 2019
-    no_comments_re = r'(?mxs)(//.*?\n|/\*.*?\*/)'
-
-    # Also note: this accepts: 'module x(arg) =' and 'function y(arg) {', both
-    # of which are incorrect syntax
-    mod_re = r'(?mxs)^\s*(?:module|function)\s+(?P<callable_name>\w+)\s*\((?P<all_args>.*?)\)\s*(?:{|=)'
-
-    # See https://github.com/SolidCode/SolidPython/issues/95; Thanks to https://github.com/Torlos
-    args_re = r'(?mxs)(?P<arg_name>\w+)(?:\s*=\s*(?P<default_val>([\w.\"\s\?:\-+\\\/*]+|\((?>[^()]|(?2))*\)|\[(?>[^\[\]]|(?2))*\])+))?(?:,|$)'
-
-    # remove all comments from SCAD code
-    scad_code_str = re.sub(no_comments_re, '', scad_code_str)
-    # get all SCAD callables
-    mod_matches = re.finditer(mod_re, scad_code_str)
-
-    for m in mod_matches:
-        callable_name = m.group('callable_name')
-        args = []
-        kwargs = []
-        all_args = m.group('all_args')
-        if all_args:
-            arg_matches = re.finditer(args_re, all_args)
-            for am in arg_matches:
-                arg_name = am.group('arg_name')
-                # NOTE: OpenSCAD's arguments to all functions are effectively
-                # optional, in contrast to Python in which all args without
-                # default values are required.
-                kwargs.append(arg_name)
-
-        callables.append({'name': callable_name, 'args': args, 'kwargs': kwargs})
+    callables = []
+    for c in modules + functions:
+        callables.append({'name': c.name, 'args': [], 'kwargs': c.parameters})
 
     return callables