[uim-commit] r1545 - branches/r5rs/sigscheme

[yamaken at freedesktop.org]

Author: yamaken
Date: 2005-09-23 03:48:44 -0700 (Fri, 23 Sep 2005)
New Revision: 1545

Modified:
   branches/r5rs/sigscheme/operations.c
Log:
* sigscheme/operations.c
  - (ScmOp_append, ScmOp_map): Add a FIXME comment


Modified: branches/r5rs/sigscheme/operations.c
===================================================================
--- branches/r5rs/sigscheme/operations.c	2005-09-23 10:28:56 UTC (rev 1544)
+++ branches/r5rs/sigscheme/operations.c	2005-09-23 10:48:44 UTC (rev 1545)
@@ -1008,6 +1008,12 @@
     return Scm_NewInt(len);
 }
 
+/*
+ * FIXME: Invalid direct cdr part referencing as lvalue. Don't assume such
+ * specific storage model. It breaks the abstract storage API. For example,
+ * base pointer + offset representation will not work under the lvalue
+ * assumption. Use SET_CDR properly.  -- YamaKen 2005-09-23
+ */
 ScmObj ScmOp_append(ScmObj args, ScmObj env)
 {
     ScmObj ret_list = SCM_NULL;
@@ -1855,6 +1861,7 @@
     return (FUNCP(obj) || CLOSUREP(obj)) ? SCM_TRUE : SCM_FALSE;
 }
 
+/* FIXME: excessive memory consumptions (reverse, vector) */
 ScmObj ScmOp_map(ScmObj map_arg, ScmObj env)
 {
     int arg_len = SCM_INT_VALUE(ScmOp_length(map_arg));