xml_reflection: First step to enable deprecation of public access

Author: EricCousineau-TRI

src/urdf_parser_py/_xml_reflection.py

@@ -0,0 +1,3 @@
+# TODO(eacousineau): Move all symbols from `.xml_reflection` into here.
+from urdf_parser_py.xml_reflection.basics import *
+from urdf_parser_py.xml_reflection.core import *

src/urdf_parser_py/xml_reflection/__init__.py

@@ -1 +1,2 @@
+# TODO(eacousineau): Deprecate public access.
 from urdf_parser_py.xml_reflection.core import *

src/urdf_parser_py/xml_reflection/basics.py

@@ -1,7 +1,28 @@
-import string
-import yaml
+# TODO(eacousineau): Deprecate public access.
 import collections
+import string
+# TODO(eacousineau): Leverage tfoote's PR.
+from xml.etree.ElementTree import ElementTree
+
 from lxml import etree
+import yaml
+
+from urdf_parser_py import _now_private_property
+
+__all__ = [
+    "xml_string",
+    "dict_sub",
+    "node_add",
+    "pfloat",
+    "xml_children",
+    "isstring",
+    "to_yaml",
+    "SelectiveReflection",
+    "YamlReflection",
+    # Backwards compatibility.
+    "etree",
+]
+
 
 def xml_string(rootXml, addHeader=True):
     # Meh
@@ -74,13 +95,15 @@ def to_yaml(obj):
 
 
 class SelectiveReflection(object):
-    def get_refl_vars(self):
+    def _get_refl_vars(self):
         return list(vars(self).keys())
 
+    get_refl_vars = _now_private_property('_get_refl_vars')
+
 
 class YamlReflection(SelectiveReflection):
     def to_yaml(self):
-        raw = dict((var, getattr(self, var)) for var in self.get_refl_vars())
+        raw = dict((var, getattr(self, var)) for var in self._get_refl_vars())
         return to_yaml(raw)
 
     def __str__(self):