From c365a26e9df84b196eb015a0c82557fd12484da1 Mon Sep 17 00:00:00 2001 From: Jay Berkenbilt Date: Sat, 30 Apr 2022 14:14:41 -0400 Subject: [PATCH] Revert "Remove QPDFObjectHandle::replaceOrRemoveKey" --- ChangeLog | 9 --------- include/qpdf/QPDFObjectHandle.hh | 4 ++++ libqpdf/QPDFObjectHandle.cc | 7 +++++++ manual/release-notes.rst | 4 ---- 4 files changed, 11 insertions(+), 13 deletions(-) diff --git a/ChangeLog b/ChangeLog index efbceb9..2f06936 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,12 +1,3 @@ -2022-04-30 Jay Berkenbilt - - * Remove QPDFObjectHandle::replaceOrRemoveKey. Call replaceKey - instead. Its functionality is identical. Note: usually I would - deprecate this for a release cycle, but the release of qpdf 11 - will require virtually all users to touch their source code, so it - actually seems better to just break this now than to put it off, - particularly since the fix so trivial. - 2022-04-29 Jay Berkenbilt * QPDFObjectHandle: for the methods insertItem, appendItem, diff --git a/include/qpdf/QPDFObjectHandle.hh b/include/qpdf/QPDFObjectHandle.hh index 48b7cda..77bef52 100644 --- a/include/qpdf/QPDFObjectHandle.hh +++ b/include/qpdf/QPDFObjectHandle.hh @@ -1068,6 +1068,10 @@ class QPDFObjectHandle QPDF_DLL QPDFObjectHandle removeKeyAndGet(std::string const& key); + // ABI: Remove in qpdf 12 + [[deprecated("use replaceKey -- it does the same thing")]] QPDF_DLL void + replaceOrRemoveKey(std::string const& key, QPDFObjectHandle const&); + // Methods for stream objects QPDF_DLL QPDFObjectHandle getDict(); diff --git a/libqpdf/QPDFObjectHandle.cc b/libqpdf/QPDFObjectHandle.cc index c08c953..76a6127 100644 --- a/libqpdf/QPDFObjectHandle.cc +++ b/libqpdf/QPDFObjectHandle.cc @@ -1334,6 +1334,13 @@ QPDFObjectHandle::removeKeyAndGet(std::string const& key) return result; } +void +QPDFObjectHandle::replaceOrRemoveKey( + std::string const& key, QPDFObjectHandle const& value) +{ + replaceKey(key, value); +} + // Stream accessors QPDFObjectHandle QPDFObjectHandle::getDict() diff --git a/manual/release-notes.rst b/manual/release-notes.rst index 61143f8..b5e4e15 100644 --- a/manual/release-notes.rst +++ b/manual/release-notes.rst @@ -64,10 +64,6 @@ For a detailed list of changes, please see the file - API: breaking changes - - Remove ``QPDFObjectHandle::replaceOrRemoveKey``. This does the - same thing as ``QPDFObjectHandle::replaceKey``. Just call - ``replaceKey`` instead. - - Remove ``QPDFAcroFormDocumentHelper::copyFieldsFromForeignPage``. This method never worked and only did something in qpdf version -- libgit2 0.21.4