* BINDING_STACK_SIZE => SPECPDL_INDEX
@ 2002-07-09 18:52 Richard Stallman
2002-07-10 8:46 ` Juanma Barranquero
0 siblings, 1 reply; 15+ messages in thread
From: Richard Stallman @ 2002-07-09 18:52 UTC (permalink / raw)
The macro BINDING_STACK_SIZE was introduced for
specpdl_ptr - specpdl, but its name is misleading.
The value is not the *size* of the stack, but rather
the position of the stack pointer within the stack.
A better name would be SPECPDL_INDEX.
Would someone please make that global replacement?
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-09 18:52 BINDING_STACK_SIZE => SPECPDL_INDEX Richard Stallman
@ 2002-07-10 8:46 ` Juanma Barranquero
2002-07-11 13:27 ` Kim F. Storm
0 siblings, 1 reply; 15+ messages in thread
From: Juanma Barranquero @ 2002-07-10 8:46 UTC (permalink / raw)
Cc: emacs-devel
On Tue, 9 Jul 2002 12:52:10 -0600 (MDT), Richard Stallman <rms@gnu.org> wrote:
> A better name would be SPECPDL_INDEX.
>
> Would someone please make that global replacement?
Done.
/L/e/k/t/u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-10 8:46 ` Juanma Barranquero
@ 2002-07-11 13:27 ` Kim F. Storm
2002-07-11 13:08 ` Juanma Barranquero
0 siblings, 1 reply; 15+ messages in thread
From: Kim F. Storm @ 2002-07-11 13:27 UTC (permalink / raw)
Cc: rms, emacs-devel
Juanma Barranquero <lektu@terra.es> writes:
> On Tue, 9 Jul 2002 12:52:10 -0600 (MDT), Richard Stallman <rms@gnu.org> wrote:
>
> > A better name would be SPECPDL_INDEX.
> >
> > Would someone please make that global replacement?
>
> Done.
For consistency, I suggest to change all occurrences of `specpdl_ptr -
specpdl' to SPECPDL_INDEX.
Also the name of the variable where we store the value
of SPECPDL_INDEX is typically named `count' which I find pretty
confusing reading the code ... "count of what?".
I'd suggest changing the name of those variables to `specpdl_index' or
just `spix' (which isn't currently used anywhere) for short.
--
Kim F. Storm <storm@cua.dk> http://www.cua.dk
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 13:27 ` Kim F. Storm
@ 2002-07-11 13:08 ` Juanma Barranquero
2002-07-11 14:48 ` Kim F. Storm
0 siblings, 1 reply; 15+ messages in thread
From: Juanma Barranquero @ 2002-07-11 13:08 UTC (permalink / raw)
Cc: rms, emacs-devel
On 11 Jul 2002 15:27:33 +0200, storm@cua.dk (Kim F. Storm) wrote:
> For consistency, I suggest to change all occurrences of `specpdl_ptr -
> specpdl' to SPECPDL_INDEX.
OK, I'll take care of it.
> I'd suggest changing the name of those variables to `specpdl_index' or
> just `spix' (which isn't currently used anywhere) for short.
"spix" seems a bit cryptic to me.
/L/e/k/t/u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 13:08 ` Juanma Barranquero
@ 2002-07-11 14:48 ` Kim F. Storm
2002-07-11 15:09 ` Juanma Barranquero
2002-07-12 17:37 ` Richard Stallman
0 siblings, 2 replies; 15+ messages in thread
From: Kim F. Storm @ 2002-07-11 14:48 UTC (permalink / raw)
Cc: Kim F. Storm, rms, emacs-devel
Juanma Barranquero <lektu@terra.es> writes:
> On 11 Jul 2002 15:27:33 +0200, storm@cua.dk (Kim F. Storm) wrote:
>
> > For consistency, I suggest to change all occurrences of `specpdl_ptr -
> > specpdl' to SPECPDL_INDEX.
>
> OK, I'll take care of it.
>
> > I'd suggest changing the name of those variables to `specpdl_index' or
> > just `spix' (which isn't currently used anywhere) for short.
>
> "spix" seems a bit cryptic to me.
Less cryptic that specpdl_index? :-)
Actually, `specpdl_index' isn't really good either, as it is not the
current index, but the saved index.
Maybe one of these is better:
old_specpdl
saved_specpdl
specpdl_save
specpdl_base
base_specpdl
specpdl_top
--
Kim F. Storm <storm@cua.dk> http://www.cua.dk
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 14:48 ` Kim F. Storm
@ 2002-07-11 15:09 ` Juanma Barranquero
2002-07-11 15:48 ` Andreas Schwab
2002-07-11 16:55 ` Stefan Monnier
2002-07-12 17:37 ` Richard Stallman
1 sibling, 2 replies; 15+ messages in thread
From: Juanma Barranquero @ 2002-07-11 15:09 UTC (permalink / raw)
Cc: Kim F. Storm, rms, emacs-devel
On 11 Jul 2002 16:48:02 +0200, no-spam@cua.dk (Kim F. Storm) wrote:
> Less cryptic that specpdl_index? :-)
Well, er... yeah, I suppose... more or less. :-)
> Actually, `specpdl_index' isn't really good either, as it is not the
> current index, but the saved index.
Hm. I've blindly made the change, but now I have second thoughts.
In alloc.c, for example, there's that code:
for (bind = specpdl; bind != specpdl_ptr; bind++)
{
mark_object (&bind->symbol);
mark_object (&bind->old_value);
}
Both specpdl and specpdl_ptr are pointers to the same structure, aren't
they? So specpdl_ptr - specpdl is really a count and not an index.
> Maybe one of these is better:
> old_specpdl
> saved_specpdl
> specpdl_save
> specpdl_base
> base_specpdl
> specpdl_top
Assuming it is really an index and not a count, any one of those is
better, yes.
/L/e/k/t/u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 15:09 ` Juanma Barranquero
@ 2002-07-11 15:48 ` Andreas Schwab
2002-07-11 16:04 ` Juanma Barranquero
2002-07-11 16:55 ` Stefan Monnier
1 sibling, 1 reply; 15+ messages in thread
From: Andreas Schwab @ 2002-07-11 15:48 UTC (permalink / raw)
Cc: Kim F. Storm, Kim F. Storm, rms, emacs-devel
Juanma Barranquero <lektu@terra.es> writes:
|> In alloc.c, for example, there's that code:
|>
|> for (bind = specpdl; bind != specpdl_ptr; bind++)
|> {
|> mark_object (&bind->symbol);
|> mark_object (&bind->old_value);
|> }
|>
|> Both specpdl and specpdl_ptr are pointers to the same structure, aren't
|> they? So specpdl_ptr - specpdl is really a count and not an index.
It is also an index into the specpdl array.
Andreas.
--
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 15:48 ` Andreas Schwab
@ 2002-07-11 16:04 ` Juanma Barranquero
0 siblings, 0 replies; 15+ messages in thread
From: Juanma Barranquero @ 2002-07-11 16:04 UTC (permalink / raw)
Cc: Kim F. Storm, rms, emacs-devel
On Thu, 11 Jul 2002 17:48:01 +0200, Andreas Schwab <schwab@suse.de> wrote:
> It is also an index into the specpdl array.
Well, yeah, you're right. This is C we're talking 'bout, after all :)
Anyway, the switch to SPECPDL_INDEX is now done. Changing the variables
(count, speccount, speccount1, post_read_count, count1 and specpdl_count)
to use a single name seems sensible. I'll do it a bit at a time.
I'll use specpdl_top, if no one opposes it.
/L/e/k/t/u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 15:09 ` Juanma Barranquero
2002-07-11 15:48 ` Andreas Schwab
@ 2002-07-11 16:55 ` Stefan Monnier
2002-07-11 17:05 ` Juanma Barranquero
1 sibling, 1 reply; 15+ messages in thread
From: Stefan Monnier @ 2002-07-11 16:55 UTC (permalink / raw)
Cc: Kim F. Storm, Kim F. Storm, rms, emacs-devel
> On 11 Jul 2002 16:48:02 +0200, no-spam@cua.dk (Kim F. Storm) wrote:
>
> > Less cryptic that specpdl_index? :-)
>
> Well, er... yeah, I suppose... more or less. :-)
>
> > Actually, `specpdl_index' isn't really good either, as it is not the
> > current index, but the saved index.
>
> Hm. I've blindly made the change, but now I have second thoughts.
Could people refrain from making such "gratuitous" pervasive changes ?
It's a pain in the ass to maintain local uncommitted changes when
you guys do that.
Stefan
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 16:55 ` Stefan Monnier
@ 2002-07-11 17:05 ` Juanma Barranquero
2002-07-11 17:08 ` Stefan Monnier
0 siblings, 1 reply; 15+ messages in thread
From: Juanma Barranquero @ 2002-07-11 17:05 UTC (permalink / raw)
Cc: Kim F. Storm, rms, emacs-devel
On Thu, 11 Jul 2002 12:55:14 -0400, "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> wrote:
> Could people refrain from making such "gratuitous" pervasive changes ?
Sorry. I changed BINDING_STACK_SIZE to SPECPDL_INDEX as per Richard's
request, and then it seemed logical to change all uses of specpdl_ptr -
specpdl to SPECPDL_INDEX.
/L/e/k/t/u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 17:05 ` Juanma Barranquero
@ 2002-07-11 17:08 ` Stefan Monnier
2002-07-12 7:45 ` Juanma Barranquero
0 siblings, 1 reply; 15+ messages in thread
From: Stefan Monnier @ 2002-07-11 17:08 UTC (permalink / raw)
Cc: Stefan Monnier, Kim F. Storm, rms, emacs-devel
> > Could people refrain from making such "gratuitous" pervasive changes ?
>
> Sorry. I changed BINDING_STACK_SIZE to SPECPDL_INDEX as per Richard's
This is fine. It's a new macro so it's OK to change it (code that uses
it is still fresh).
> request, and then it seemed logical to change all uses of specpdl_ptr -
> specpdl to SPECPDL_INDEX.
This might be a good change, so I can live with it: it's not gratuitous.
But the count -> index|spix|... switch is just plain silly and will only
make my life worse.
Stefan
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 17:08 ` Stefan Monnier
@ 2002-07-12 7:45 ` Juanma Barranquero
2002-07-12 10:16 ` Kim F. Storm
0 siblings, 1 reply; 15+ messages in thread
From: Juanma Barranquero @ 2002-07-12 7:45 UTC (permalink / raw)
Cc: Kim F. Storm, rms, emacs-devel
On Thu, 11 Jul 2002 13:08:58 -0400, "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> wrote:
> But the count -> index|spix|... switch is just plain silly
Hmm... Not really. There are six or seven different names for the same
concept through the code. Unifying them would be useful, if only to make
understanding simpler for newcomers (like me).
But I suppose others will have to decide what to do in that case :)
/L/e/k/t/u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-12 7:45 ` Juanma Barranquero
@ 2002-07-12 10:16 ` Kim F. Storm
0 siblings, 0 replies; 15+ messages in thread
From: Kim F. Storm @ 2002-07-12 10:16 UTC (permalink / raw)
Cc: Stefan Monnier, Kim F. Storm, rms, emacs-devel
Juanma Barranquero <lektu@terra.es> writes:
> On Thu, 11 Jul 2002 13:08:58 -0400, "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> wrote:
>
> > But the count -> index|spix|... switch is just plain silly
Changes which can improve the maintainablility of the code are not silly.
You may not want them, and we may decide against them, but that doesn't
make them silly.
It seems that some emacs hackers have a habit of keeping big thunks of
changes locally for a long time... That's fine, but doing so should
not prevent others from maintaining the code in CVS.
>
> Hmm... Not really. There are six or seven different names for the same
> concept through the code. Unifying them would be useful, if only to make
> understanding simpler for newcomers (like me).
Exactly my experience.
>
> But I suppose others will have to decide what to do in that case :)
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: BINDING_STACK_SIZE => SPECPDL_INDEX
2002-07-11 14:48 ` Kim F. Storm
2002-07-11 15:09 ` Juanma Barranquero
@ 2002-07-12 17:37 ` Richard Stallman
2002-07-13 3:20 ` Juanma Barranquero
1 sibling, 1 reply; 15+ messages in thread
From: Richard Stallman @ 2002-07-12 17:37 UTC (permalink / raw)
Cc: lektu, storm, emacs-devel
Please don't rename the local variables called `count'.
That is a lot of change for very little benefit.
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2002-07-13 3:20 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-07-09 18:52 BINDING_STACK_SIZE => SPECPDL_INDEX Richard Stallman
2002-07-10 8:46 ` Juanma Barranquero
2002-07-11 13:27 ` Kim F. Storm
2002-07-11 13:08 ` Juanma Barranquero
2002-07-11 14:48 ` Kim F. Storm
2002-07-11 15:09 ` Juanma Barranquero
2002-07-11 15:48 ` Andreas Schwab
2002-07-11 16:04 ` Juanma Barranquero
2002-07-11 16:55 ` Stefan Monnier
2002-07-11 17:05 ` Juanma Barranquero
2002-07-11 17:08 ` Stefan Monnier
2002-07-12 7:45 ` Juanma Barranquero
2002-07-12 10:16 ` Kim F. Storm
2002-07-12 17:37 ` Richard Stallman
2002-07-13 3:20 ` Juanma Barranquero
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.