Last thoughts about the parser.

2 files changed, 7 insertions, 5 deletions

diff --git a/kronos/parser.py b/kronos/parser.py
index 8d93391..f0543f9 100644
--- a/kronos/parser.py
+++ b/kronos/parser.py
@@ -24,8 +24,8 @@ def parse_activity(text):
         description = description.strip()
 
         tokens = description.split(' ')
-        tags = parse_tags(tokens)
-        description = strip_tags(tokens)
+        tags = _parse_tags(tokens)
+        description = _strip_tags(tokens)
 
     category = None
     if '@' in text:
@@ -38,9 +38,10 @@ def parse_activity(text):
     return activity
 
 
-def parse_tags(tokens):
+def _parse_tags(tokens):
     return [token.lstrip('#') for token in tokens if token.startswith('#')]
 
 
-def strip_tags(tokens):
-    return ' '.join([token for token in tokens if not token.startswith('#')])
+def _strip_tags(tokens):
+    words = [token for token in tokens if not token.startswith('#')]
+    return ' '.join(words)
diff --git a/kronos/tests/test_parser.py b/kronos/tests/test_parser.py
index 015ae06..8a82a65 100644
--- a/kronos/tests/test_parser.py
+++ b/kronos/tests/test_parser.py
@@ -61,6 +61,7 @@ class TestWhenParsingWithTags(object):
         assert_equals(self.results.description, "Some cool stuff!")
 
 
+# TODO: Implement this functionality when I need it.
 class TestWhenParsingWithTimeOffset(object):
 
     def test_should_understand_minutes(self):