diff options
| author | Robin Haberkorn <robin.haberkorn@googlemail.com> | 2025-03-21 13:26:01 +0300 |
|---|---|---|
| committer | Robin Haberkorn <robin.haberkorn@googlemail.com> | 2025-03-21 13:40:52 +0300 |
| commit | 33f71654136014bac094babaaa81d91245fdd24c (patch) | |
| tree | 2090722fc330ded11fb00321a8afd7e0e81bafaa /src/qreg.h | |
| parent | 121cca61bedf24ba428e019f23b3bbb7160ce162 (diff) | |
| download | sciteco-33f71654136014bac094babaaa81d91245fdd24c.tar.gz | |
fixed rubout of Q-Register specifications
* This was a regression introduced by 257a0bf128e109442dce91c4aaa1d97fed17ad1a.
* The undo token that frees newly allocated teco_machine_qregspec_t must actually
reset the pointer as well since any subsequent token, pushed by teco_undo_qregspec_own(),
will expect a valid pointer.
* Could have been done via
ctx->expectqreg = NULL;
teco_undo_qregspec_own(ctx->expectqreg);
but using a special clear function requires less memory and is easier to understand.
* Added test case. This wouldn't always crash, but should definitely show up in Valgrind.
Diffstat (limited to 'src/qreg.h')
| -rw-r--r-- | src/qreg.h | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -252,7 +252,7 @@ void teco_machine_qregspec_free(teco_machine_qregspec_t *ctx); G_DEFINE_AUTOPTR_CLEANUP_FUNC(teco_machine_qregspec_t, teco_machine_qregspec_free); /** @memberof teco_machine_qregspec_t */ -void undo__teco_machine_qregspec_free(teco_machine_qregspec_t *); +void undo__teco_machine_qregspec_clear(teco_machine_qregspec_t **); TECO_DECLARE_UNDO_OBJECT(qregspec, teco_machine_qregspec_t *); #define teco_undo_qregspec_own(VAR) \ |