From: Robert Kessl <kesslr@centrum.cz>
Date: Wed, 30 Jul 2014 08:15:49 +0000 (+0200)
Subject: tableprinter: code cleanup
X-Git-Tag: v6.1~3^2~20
X-Git-Url: http://mj.ucw.cz/gitweb/?a=commitdiff_plain;h=8b9d69756e344982623e10a17a6a35933d0c3a21;p=libucw.git

tableprinter: code cleanup
---

diff --git a/ucw/table.h b/ucw/table.h
index 2fdadbbf..ef3c30ab 100644
--- a/ucw/table.h
+++ b/ucw/table.h
@@ -345,8 +345,6 @@ void table_set_col_order(struct table *tbl, const struct table_col_instance *col
  * <col-name> is a string that does not contain comma ',' or '[',']' brackets
  *
  * <col-opt> is currently only one string without commas. In the future the format can be <str1>,<str2>,... .
- *
- * FIXME In the future, we should allow <col-opt> to be a comma(,) separated list of identifiers
  **/
 const char *table_set_col_order_by_name(struct table *tbl, const char *col_order);
 
diff --git a/ucw/xtypes-extra.c b/ucw/xtypes-extra.c
index 589ac2cb..c0c31a83 100644
--- a/ucw/xtypes-extra.c
+++ b/ucw/xtypes-extra.c
@@ -15,12 +15,6 @@
 
 #include <ucw/xtypes-extra.h>
 
-// FIXME: I seriously doubt there is any good reason for keeping
-// these types separated from the generic xtype machinery. There
-// is nothing special in them, which would be tightly connected
-// to the table printer. Especially as they are already tested
-// by xtypes-test.c.  --mj
-
 /** xt_size **/
 
 static const struct unit_definition xt_size_units[] = {