From 3c7186e871ece5a0ffae02338cfe3081c7716336 Mon Sep 17 00:00:00 2001 From: Akira Kakuto Date: Sat, 17 Apr 2021 22:31:16 +0000 Subject: harfbuzz 2.8.0 git-svn-id: svn://tug.org/texlive/trunk@58898 c570f23f-e606-0410-a88d-b1316a301751 --- Build/source/libs/harfbuzz/harfbuzz-src/src/hb-serialize.hh | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'Build/source/libs/harfbuzz/harfbuzz-src/src/hb-serialize.hh') diff --git a/Build/source/libs/harfbuzz/harfbuzz-src/src/hb-serialize.hh b/Build/source/libs/harfbuzz/harfbuzz-src/src/hb-serialize.hh index 4566153a593..fe29bdf96ee 100644 --- a/Build/source/libs/harfbuzz/harfbuzz-src/src/hb-serialize.hh +++ b/Build/source/libs/harfbuzz/harfbuzz-src/src/hb-serialize.hh @@ -256,10 +256,11 @@ struct hb_serialize_context_t packed.push (obj); - if (unlikely (packed.in_error ())) { - // obj wasn't successfully added to packed, so clean it up otherwise it's - // links will be leaked. - propagate_error (packed); + if (unlikely (!propagate_error (packed))) + { + /* Obj wasn't successfully added to packed, so clean it up otherwise its + * links will be leaked. When we use constructor/destructors properly, we + * can remove these. */ obj->fini (); return 0; } @@ -523,7 +524,7 @@ struct hb_serialize_context_t template void assign_offset (const object_t* parent, const object_t::link_t &link, unsigned offset) { - auto &off = * ((BEInt *) (parent->head + link.position)); + auto &off = * ((BEInt *) (parent->head + link.position)); assert (0 == off); check_assign (off, offset); } -- cgit v1.2.3