From: Rob Browning <rlb@defaultvalue.org>
Cc: Dirk Herrmann <dirk@sallust.ida.ing.tu-bs.de>, guile-devel@gnu.org
Subject: Re: GC rewrite, first version.
Date: Wed, 31 Jul 2002 16:15:11 -0500 [thread overview]
Message-ID: <87eldja9b4.fsf@raven.i.defaultvalue.org> (raw)
In-Reply-To: <15688.9798.978583.867042@meddo.cs.uu.nl> (Han-Wen Nienhuys's message of "Wed, 31 Jul 2002 20:02:46 +0200")
Han-Wen Nienhuys <hanwen@cs.uu.nl> writes:
> Personally, I view source code as a form of prose. It should be
> easily readable. Hence I prefer variable names that can just be read
> as an english expression, with the prefix scm_ for delimiting our
> namespace, perhaps with an abbreviation here or there to keep the
> lines within my editor window. Typically, the most generic word
> (always a noun), eg number, size, string, comes last
Overall I agree, though I will say that all other things being equal,
I tend to feel that consistency within the codebase, up to a certain
point, improves readability (and managability wrt finding things via
the normal tools like grep, etc.) more than having some files with the
"perfect style" and others without would.
> scm_heap_segment_table_size
> scm_formals_str(ing)
>
> in stead of
>
> scm_n_heap_segs
> scm_s_formals
I consider the latter uglier, and wouldn't choose it myself, but I
could manage with either if the convention were consistently used.
> And for the rest, I think that GUILE 1.6 should be released.
Agreed. I've fixed the remaining issues, and 1.5.7 has been ready for
upload since last week, but I've been waiting for alpha.gnu.org to
become available again.
> As an aside, if no-one objects to it, I would like to merge the new GC
> code coming weekend.
No opinion -- I'd rather have the people who know the GC better
comment on that, but I definitely appreciate your efforts in trying to
clean things up. Seems like the gc and eval are two of the
(understandably) harder segments of code to get a handle on when you
start messing with guile, so the more we can make these transparent
without appreciably harming our performance or functionality, the
better.
--
Rob Browning
rlb @defaultvalue.org, @linuxdevel.com, and @debian.org
Previously @cs.utexas.edu
GPG=1C58 8B2C FB5E 3F64 EA5C 64AE 78FE E5FE F0CB A0AD
_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://mail.gnu.org/mailman/listinfo/guile-devel
next prev parent reply other threads:[~2002-07-31 21:15 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-07-27 22:54 GC rewrite, first version Han-Wen
2002-07-28 16:40 ` Rob Browning
2002-07-29 20:11 ` Dirk Herrmann
2002-07-29 21:04 ` Rob Browning
2002-07-29 22:05 ` Han-Wen
2002-07-31 17:03 ` Dirk Herrmann
2002-07-31 18:02 ` Han-Wen Nienhuys
2002-07-31 21:15 ` Rob Browning [this message]
2002-08-01 9:20 ` Release Guile, now ;-) [was:] " rm
2002-08-01 16:27 ` Rob Browning
2002-08-01 16:44 ` rm
2002-08-01 18:37 ` Sergey Poznyakoff
2002-08-01 22:21 ` Rob Browning
2002-08-02 6:09 ` Sergey Poznyakoff
2002-08-02 14:36 ` Rob Browning
2002-08-02 17:29 ` Sergey Poznyakoff
2002-08-02 18:10 ` Bruce Korb
2002-08-02 19:50 ` Rob Browning
2002-08-03 7:13 ` Sergey Poznyakoff
2002-08-04 20:43 ` Bruce Korb
2002-08-04 20:57 ` Sergey Poznyakoff
2002-08-01 22:40 ` Rob Browning
2002-08-02 9:35 ` rm
2002-08-02 11:59 ` rm
2002-08-02 15:00 ` Rob Browning
2002-08-02 14:50 ` Rob Browning
2002-08-01 9:59 ` Han-Wen Nienhuys
2002-08-01 8:46 ` Marius Vollmer
2002-07-31 18:46 ` Neil Jerram
2002-08-01 9:58 ` Han-Wen Nienhuys
2002-07-28 16:51 ` Michael Livshin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87eldja9b4.fsf@raven.i.defaultvalue.org \
--to=rlb@defaultvalue.org \
--cc=dirk@sallust.ida.ing.tu-bs.de \
--cc=guile-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).