From: Thomas Mansencal <thomas.mansencal@gmail.com>
Date: Wed, 29 Jul 2015 11:54:59 +0000 (+1200)
Subject: Add various TODOs.
X-Git-Url: http://users.mur.at/ms/git/gitweb/?p=OpenColorIO-Configs.git;a=commitdiff_plain;h=918520c7cc16aa549e2af0f57b630905063bad19

Add various TODOs.
---

diff --git a/aces_1.0.0/python/aces_ocio/colorspaces/aces.py b/aces_1.0.0/python/aces_ocio/colorspaces/aces.py
index e16e5fe..c3e725e 100644
--- a/aces_1.0.0/python/aces_ocio/colorspaces/aces.py
+++ b/aces_1.0.0/python/aces_ocio/colorspaces/aces.py
@@ -737,6 +737,7 @@ def create_ACES_LMT(lmt_name,
         ctls = [os.path.join(aces_ctl_directory,
                              lmt_values['transformCTLInverse']),
                 shaper_from_aces_ctl % aces_ctl_directory]
+        # TODO: Investigate unresolved `odt_name` reference.
         lut = 'Inverse.%s.%s.spi3d' % (odt_name, shaper_name)
 
         lut = sanitize(lut)
diff --git a/aces_1.0.0/python/aces_ocio/process.py b/aces_1.0.0/python/aces_ocio/process.py
index 8312ccd..76dd754 100755
--- a/aces_1.0.0/python/aces_ocio/process.py
+++ b/aces_1.0.0/python/aces_ocio/process.py
@@ -41,9 +41,13 @@ def read_text(text_file):
          Return value description.
     """
 
-    if text_file != '':
-        with open(text_file, 'rb') as fp:
-            text = (fp.read())
+    # TODO: Investigate if check is needed.
+    if not text_file:
+        return
+
+    with open(text_file, 'rb') as fp:
+        text = (fp.read())
+
     return text
 
 
@@ -62,9 +66,13 @@ def write_text(text, text_file):
          Return value description.
     """
 
-    if text_file != '':
-        with open(text_file, 'wb') as fp:
-            fp.write(text)
+    # TODO: Investigate if check is needed.
+    if not text_file:
+        return
+
+    with open(text_file, 'wb') as fp:
+        fp.write(text)
+
     return text