summaryrefslogtreecommitdiff
path: root/source/fitz/font.c
diff options
context:
space:
mode:
authorTor Andersson <tor.andersson@artifex.com>2015-02-17 10:30:59 +0100
committerTor Andersson <tor.andersson@artifex.com>2015-02-17 18:06:11 +0100
commit72679561cddc6b2586e596f62492b79dcf9f118d (patch)
tree25d156aacc2d2ac3e33170ec42b6501edb9b6754 /source/fitz/font.c
parent6cc97e85489f5e4e39aa185d17ad5e35b09dddb5 (diff)
downloadmupdf-72679561cddc6b2586e596f62492b79dcf9f118d.tar.xz
Add helper functions to keep/drop reference counts with locking.
Add locks around fz_path and fz_text reference counting.
Diffstat (limited to 'source/fitz/font.c')
-rw-r--r--source/fitz/font.c29
1 files changed, 7 insertions, 22 deletions
diff --git a/source/fitz/font.c b/source/fitz/font.c
index e35f01fa..5f2f85e1 100644
--- a/source/fitz/font.c
+++ b/source/fitz/font.c
@@ -74,12 +74,7 @@ fz_new_font(fz_context *ctx, const char *name, int use_glyph_bbox, int glyph_cou
fz_font *
fz_keep_font(fz_context *ctx, fz_font *font)
{
- if (!font)
- return NULL;
- fz_lock(ctx, FZ_LOCK_ALLOC);
- font->refs ++;
- fz_unlock(ctx, FZ_LOCK_ALLOC);
- return font;
+ return fz_keep_imp(ctx, font, &font->refs);
}
static void
@@ -119,12 +114,9 @@ void
fz_drop_font(fz_context *ctx, fz_font *font)
{
int fterr;
- int i, drop;
+ int i;
- fz_lock(ctx, FZ_LOCK_ALLOC);
- drop = (font && --font->refs == 0);
- fz_unlock(ctx, FZ_LOCK_ALLOC);
- if (!drop)
+ if (!fz_drop_imp(ctx, font, &font->refs))
return;
free_resources(ctx, font);
@@ -214,23 +206,16 @@ void fz_new_font_context(fz_context *ctx)
fz_font_context *
fz_keep_font_context(fz_context *ctx)
{
- if (!ctx || !ctx->font)
+ if (!ctx)
return NULL;
- fz_lock(ctx, FZ_LOCK_ALLOC);
- ctx->font->ctx_refs++;
- fz_unlock(ctx, FZ_LOCK_ALLOC);
- return ctx->font;
+ return fz_keep_imp(ctx, ctx->font, &ctx->font->ctx_refs);
}
void fz_drop_font_context(fz_context *ctx)
{
- int drop;
- if (!ctx || !ctx->font)
+ if (!ctx)
return;
- fz_lock(ctx, FZ_LOCK_ALLOC);
- drop = --ctx->font->ctx_refs;
- fz_unlock(ctx, FZ_LOCK_ALLOC);
- if (drop == 0)
+ if (fz_drop_imp(ctx, ctx->font, &ctx->font->ctx_refs))
fz_free(ctx, ctx->font);
}