Finishing touches

Signed-off-by: Jacob Hrbek <kreyren@rixotstudio.cz>

TAGS.org

@@ -23,7 +23,7 @@ Which in practice might be used as:
 - DNM = Do Not Merge - Usage in merge/pull requests blocks syncing the code, used to tag code that needs to be addressed before merge can happen
 - PROD/PRODUCTION = Code that should be considered prior to it's usage in production environment
 - TRANSLATE = Needs to be translated
-
+- WTF = Code that is not fully understood what it is doing
 
 ** Repository-wide tags
 

src/potato/makevars.scm

@@ -52,7 +52,7 @@
 		      "environment"
 		      "built-in"))
 
-;; WTF(Krey): No fucking idea.. something with ASCII? If ASCII is supported?
+;; WTF(Krey): No idea.. something with ASCII? If ASCII is supported?
 (define %ascii? #f)
 
 ;; WTF(Krey)

src/potato/parse.scm

@@ -33,16 +33,16 @@
 
 ;; A makefile can contain rules, macro definitions, include lines, and comments.
 
-;; FIXME-DOCS(Krey)
 (define (parse filename)
+  "FIXME-DOCS"
   (with-input-from-file filename _eval #:guess-encoding #t))
 
-;; DNM-SECURITY(Krey): Check very carefully
-;; FIXME-DOCS(Krey)
 (define (_eval)
+  "FIXME-DOCS"
   (let ((filenames #f)
 	(ignoring #f)
 	(commands '()))
+    ;; FIXME-QA(Krey): `(while #t` is not robust enough, specify the logic to lesser the chance of infinite loops
     (while #t
       (receive (line nlines)
 	  (read-line-handle-escaped-newline)

tests/zz-README.org

@@ -1 +1,4 @@
 Directory storing tests
+
+
+NOTE-QA(Krey): Tests were ignored in the QA, decided that they should be rewritten once we finish refactor of the project