From f66bcef797ad732c294ffc8168bbfe50e86ffca7 Mon Sep 17 00:00:00 2001 From: Robin Watts Date: Wed, 14 Sep 2016 18:02:35 +0100 Subject: Don't report addRef/dropRef events to Memento twice. We call Memento_addRef etc in fz_keep_impXX functions, so don't call them in the callers too. --- source/fitz/store.c | 4 ---- 1 file changed, 4 deletions(-) (limited to 'source/fitz') diff --git a/source/fitz/store.c b/source/fitz/store.c index f8feb591..cad027bc 100644 --- a/source/fitz/store.c +++ b/source/fitz/store.c @@ -60,8 +60,6 @@ fz_keep_storable(fz_context *ctx, const fz_storable *sc) * sanely throughout the code. */ fz_storable *s = (fz_storable *)sc; - if (s && s->refs > 0) - (void)Memento_takeRef(s); return fz_keep_imp(ctx, s, &s->refs); } @@ -79,8 +77,6 @@ fz_drop_storable(fz_context *ctx, const fz_storable *sc) this method. So we can simply drop the storable object itself without any operations on the fz_store. */ - if (s && s->refs > 0) - (void)Memento_dropRef(s); if (fz_drop_imp(ctx, s, &s->refs)) s->drop(ctx, s); } -- cgit v1.2.3