Add another FIXME comment

13 years ago · f5297dfc25
parent 35fca0f6af
commit f5297dfc25
1 changed files with 3 additions and 0 deletions
--- a/libraries/libmdb/mdb.c
+++ b/libraries/libmdb/mdb.c
@ -226,6 +226,9 @@ typedef struct MDB_ppage {					/* ordered list of pages */
 } MDB_ppage;
 SLIST_HEAD(page_stack, MDB_ppage);

+/* FIXME: tree depth is mostly bounded, we should just
+ * use a fixed array and avoid malloc/pointer chasing
+ */
 #define CURSOR_EMPTY(c)		 SLIST_EMPTY(&(c)->mc_stack)
 #define CURSOR_TOP(c)		 SLIST_FIRST(&(c)->mc_stack)
 #define CURSOR_POP(c)		 SLIST_REMOVE_HEAD(&(c)->mc_stack, mp_entry)