From 33ebb1f9e0b4ea4f8f9f464127c75ed146e6e6df Mon Sep 17 00:00:00 2001
From: Pavel Kirienko <pavel.kirienko@gmail.com>
Date: Sun, 2 Mar 2014 21:26:14 +0400
Subject: [PATCH] parse_namespace() parses one namespace at a time

---
 pyuavcan/pyuavcan/dsdl/__init__.py |  2 +-
 pyuavcan/pyuavcan/dsdl/parser.py   | 19 +++++++++----------
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/pyuavcan/pyuavcan/dsdl/__init__.py b/pyuavcan/pyuavcan/dsdl/__init__.py
index f364dd0624..c9399f6af4 100644
--- a/pyuavcan/pyuavcan/dsdl/__init__.py
+++ b/pyuavcan/pyuavcan/dsdl/__init__.py
@@ -2,7 +2,7 @@
 # Copyright (C) 2014 Pavel Kirienko <pavel.kirienko@gmail.com>
 #
 
-from .parser import Parser, parse_namespaces, \
+from .parser import Parser, parse_namespace, \
                     Type, PrimitiveType, ArrayType, CompoundType, \
                     Attribute, Field, Constant
 
diff --git a/pyuavcan/pyuavcan/dsdl/parser.py b/pyuavcan/pyuavcan/dsdl/parser.py
index 376c987c05..f68000bf96 100644
--- a/pyuavcan/pyuavcan/dsdl/parser.py
+++ b/pyuavcan/pyuavcan/dsdl/parser.py
@@ -484,18 +484,17 @@ def validate_data_struct_len(t):
                  'Max data structure length is invalid: %d bits, %d bytes', bitlen, bytelen)
 
 
-def parse_namespaces(directory_list):
+def parse_namespace(source_dir, search_dirs):
     def walk():
         import fnmatch
         from functools import partial
         def on_walk_error(directory, ex):
             raise DsdlException('OS error in [%s]: %s' % (directory, str(ex))) from ex
-        for directory in directory_list:
-            walker = os.walk(directory, onerror=partial(on_walk_error, directory), followlinks=True)
-            for root, _dirnames, filenames in walker:
-                for filename in fnmatch.filter(filenames, '*.uavcan'):
-                    filename = os.path.join(root, filename)
-                    yield filename
+        walker = os.walk(source_dir, onerror=partial(on_walk_error, source_dir), followlinks=True)
+        for root, _dirnames, filenames in walker:
+            for filename in fnmatch.filter(filenames, '*.uavcan'):
+                filename = os.path.join(root, filename)
+                yield filename
 
     all_default_dtid = {}  # (kind, dtid) : filename
     def ensure_unique_dtid(t, filename):
@@ -508,7 +507,7 @@ def parse_namespaces(directory_list):
             error('Default data type ID collision: [%s] [%s]', first, second)
         all_default_dtid[key] = filename
 
-    parser = Parser(directory_list)
+    parser = Parser([source_dir] + search_dirs)
     output_types = []
     for filename in walk():
         t = parser.parse(filename)
@@ -527,10 +526,10 @@ if __name__ == '__main__':
         test_dir = os.path.normpath(test_dir)
 #         parser = Parser([os.path.join(test_dir, 'root_a'), os.path.join(test_dir, 'root_b')])
 #         t = parser.parse(os.path.join(test_dir, 'root_a', 'ns1', 'ns9', '425.BeginFirmwareUpdate.uavcan'))
-        t = parse_namespaces([os.path.join(test_dir, 'root_a'), os.path.join(test_dir, 'root_b')])
+        t = parse_namespace(os.path.join(test_dir, 'root_a'), [os.path.join(test_dir, 'root_b')])
         print(len(t))
     else:
-        t = parse_namespaces(sys.argv[1:])
+        t = parse_namespace(sys.argv[1], sys.argv[2:])
         print(len(t))
 #         search_dirs = sys.argv[1:-1]
 #         filename = sys.argv[-1]