[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Reference leak in `iprin1 ()'
From: |
Neil Jerram |
Subject: |
Re: [PATCH] Reference leak in `iprin1 ()' |
Date: |
Sat, 12 Nov 2005 09:23:06 +0000 |
User-agent: |
Gnus/5.1007 (Gnus v5.10.7) Emacs/21.4 (gnu/linux) |
address@hidden (Ludovic Courtès) writes:
> * print.c (EXIT_NESTED_DATA): Before popping from the stack,
> reset the value at its top. This fixes a reference leak.
Nice work, but it looks to me that PUSH_REF sets the value of the
(pstate->top)th element _before_ incrementing pstate->top. So
shouldn't your fix do the decrement first and then set the slot to
undefined?
(Note that your fix will still prevent most reference leaks, just not
the outermost one. So that's why it may appear to be working
already.)
Also, is the () -> (void) change strictly related?
Neil
- [PATCH] Marking weak alist vectors, Ludovic Courtès, 2005/11/08
- Re: [PATCH] Marking weak alist vectors, Marius Vollmer, 2005/11/08
- Re: [PATCH] Marking weak alist vectors, Han-Wen Nienhuys, 2005/11/09
- Re: [PATCH] Marking weak alist vectors, Ludovic Courtès, 2005/11/09
- [PATCH] Reference leak in `iprin1 ()', Ludovic Courtès, 2005/11/10
- Re: [PATCH] Reference leak in `iprin1 ()',
Neil Jerram <=
- Re: [PATCH] Reference leak in `iprin1 ()', Ludovic Courtès, 2005/11/14
- Re: [PATCH] Reference leak in `iprin1 ()', Neil Jerram, 2005/11/16
- Re: [PATCH] Reference leak in `iprin1 ()', Ludovic Courtès, 2005/11/17
- Re: [PATCH] Reference leak in `iprin1 ()', Neil Jerram, 2005/11/17
- Re: [PATCH] Marking weak alist vectors, #2, Ludovic Courtès, 2005/11/23
- Re: [PATCH] Marking weak alist vectors, #2, Han-Wen Nienhuys, 2005/11/23
- Re: [PATCH] Marking weak alist vectors, #2, Ludovic Courtès, 2005/11/24
- Re: [PATCH] Marking weak alist vectors, #2, Kevin Ryde, 2005/11/25
[PATCH] Fixing `gc-live-object-stats', Ludovic Courtès, 2005/11/17