This is the mail archive of the guile@sourceware.cygnus.com mailing list for the Guile project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

guile bugs?


Hello,

Are these Guile's bugs?

* There is no line "#include <libguile/validate.h>" in libguile.h

* Why not use scm_assq in set-object-property! ?

--- objprop.c	2000/01/18 11:24:03	1.11
+++ objprop.c	2000/03/14 08:49:55
@@ -102,7 +102,7 @@
   SCM assoc;
   h = scm_hashq_create_handle_x (scm_object_whash, obj, SCM_EOL);
   SCM_DEFER_INTS;
-  assoc = scm_assoc (key, SCM_CDR (h));
+  assoc = scm_assq (key, SCM_CDR (h));
   if (SCM_NIMP (assoc))
     SCM_SETCDR (assoc, val);
   else

* guile-doc-snarf produces

(acons key value alist)
Adds a new key-value pair to @var{alist}.  A new pair is\n"
created whose car is @var{key} and whose cdr is @var{value}, and the\n"
pair is consed onto @var{alist}, and the new list is returned.  This\n"
function is @emph{not} destructive; @var{alist} is not modified.
^A[alist.c:61]

I guess the \n" at the end of each line is unnecessary, isn't it?
I also think \n is not needed in the C files either since doc-strings
will never be processed by the C compiler.


I sent the upper two to bug-guile, but I had no response...

Thanks,
Keisuke Nishida

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]