unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
* [PATCH] Static allocation of GOOPS' SMOB and port class arrays
@ 2009-01-18 20:16 Ludovic Courtès
  2009-02-02 23:10 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2009-01-18 20:16 UTC (permalink / raw)
  To: guile-devel

Hello!

In the BDW-GC branch, I committed these two pairs of patches:

  http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=c891a40e9fc5f718bfaf6e70f8fd0b19311d14a6
  http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=474554694f5618be238054e736540f567a0e02a0

  http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=0953b549464e5bf1433e7f0b8a49ac4b464c3c11
  http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=63385df2fdc2f9bc70804383f071f2fb74743a86

The first of each pair defines constants for the maximum number of
SMOB/port types and uses them, which clarifies the code.

The second changes `goops.c' so that the `scm_{smob,port}_class' arrays
are statically allocated.

If nobody objects in the next few days, I'll commit them to `master'.

Thanks,
Ludo'.





^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Static allocation of GOOPS' SMOB and port class arrays
  2009-01-18 20:16 [PATCH] Static allocation of GOOPS' SMOB and port class arrays Ludovic Courtès
@ 2009-02-02 23:10 ` Ludovic Courtès
  2009-02-03 23:11   ` Neil Jerram
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2009-02-02 23:10 UTC (permalink / raw)
  To: guile-devel

ludo@gnu.org (Ludovic Courtès) writes:

> In the BDW-GC branch, I committed these two pairs of patches:
>
>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=c891a40e9fc5f718bfaf6e70f8fd0b19311d14a6
>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=474554694f5618be238054e736540f567a0e02a0
>
>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=0953b549464e5bf1433e7f0b8a49ac4b464c3c11
>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=63385df2fdc2f9bc70804383f071f2fb74743a86
>
> The first of each pair defines constants for the maximum number of
> SMOB/port types and uses them, which clarifies the code.
>
> The second changes `goops.c' so that the `scm_{smob,port}_class' arrays
> are statically allocated.
>
> If nobody objects in the next few days, I'll commit them to `master'.

Pushed.

Ludo'.





^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Static allocation of GOOPS' SMOB and port class arrays
  2009-02-02 23:10 ` Ludovic Courtès
@ 2009-02-03 23:11   ` Neil Jerram
  0 siblings, 0 replies; 3+ messages in thread
From: Neil Jerram @ 2009-02-03 23:11 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guile-devel

ludo@gnu.org (Ludovic Courtès) writes:

> ludo@gnu.org (Ludovic Courtès) writes:
>
>> In the BDW-GC branch, I committed these two pairs of patches:
>>
>>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=c891a40e9fc5f718bfaf6e70f8fd0b19311d14a6
>>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=474554694f5618be238054e736540f567a0e02a0
>>
>>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=0953b549464e5bf1433e7f0b8a49ac4b464c3c11
>>   http://git.savannah.gnu.org/gitweb/?p=guile.git;a=commitdiff;h=63385df2fdc2f9bc70804383f071f2fb74743a86
>>
>> The first of each pair defines constants for the maximum number of
>> SMOB/port types and uses them, which clarifies the code.
>>
>> The second changes `goops.c' so that the `scm_{smob,port}_class' arrays
>> are statically allocated.
>>
>> If nobody objects in the next few days, I'll commit them to `master'.
>
> Pushed.

Somewhat belatedly... these changes look good to me.

     Neil




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-02-03 23:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-18 20:16 [PATCH] Static allocation of GOOPS' SMOB and port class arrays Ludovic Courtès
2009-02-02 23:10 ` Ludovic Courtès
2009-02-03 23:11   ` Neil Jerram

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).