core: Un-deprecate gpgme_data_rewind.
authorWerner Koch <wk@gnupg.org>
Thu, 2 Feb 2017 08:50:47 +0000 (09:50 +0100)
committerWerner Koch <wk@gnupg.org>
Thu, 2 Feb 2017 08:53:48 +0000 (09:53 +0100)
* src/gpgme.h.in (gpgme_data_rewind): Un-deprecate.
* src/data-compat.c (gpgme_data_rewind): Move to ...
* src/data.c (gpgme_data_rewind): here.
--

That function is very convenient because it is required a lot with
memory streams.  It also documents the intention of the caller better
than gpgme_data_seek with its addition parameters and the need to map
system errors.  Thus it does not make sense to make it a first class
citizen again.

Signed-off-by: Werner Koch <wk@gnupg.org>
NEWS
src/data-compat.c
src/data.c
src/gpgme.h.in

diff --git a/NEWS b/NEWS
index 0fca77a..4405f6c 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -12,6 +12,7 @@ Noteworthy changes in version 1.8.1 (unreleased)
  cpp: Key::UserID::revoke()                         NEW.
  cpp: Key::addUid()                                 NEW.
  qt: CryptoConfig::stringValueList()                NEW.
+ gpgme_data_rewind                                  UN-DEPRECATE.
 
 
 Noteworthy changes in version 1.8.0 (2016-11-16)
index 5c7d543..87eaeef 100644 (file)
@@ -239,16 +239,3 @@ gpgme_data_new_with_read_cb (gpgme_data_t *r_dh,
   (*r_dh)->data.old_user.handle = read_cb_value;
   return TRACE_ERR (0);
 }
-
-\f
-gpgme_error_t
-gpgme_data_rewind (gpgme_data_t dh)
-{
-  gpgme_error_t err;
-  TRACE_BEG (DEBUG_DATA, "gpgme_data_rewind", dh);
-
-  err = ((gpgme_data_seek (dh, 0, SEEK_SET) == -1)
-         ? gpg_error_from_syserror () : 0);
-
-  return TRACE_ERR (err);
-}
index 6964246..e4e9ee3 100644 (file)
@@ -158,6 +158,20 @@ gpgme_data_seek (gpgme_data_t dh, gpgme_off_t offset, int whence)
 }
 
 
+/* Convenience function to do a gpgme_data_seek (dh, 0, SEEK_SET).  */
+gpgme_error_t
+gpgme_data_rewind (gpgme_data_t dh)
+{
+  gpgme_error_t err;
+  TRACE_BEG (DEBUG_DATA, "gpgme_data_rewind", dh);
+
+  err = ((gpgme_data_seek (dh, 0, SEEK_SET) == -1)
+         ? gpg_error_from_syserror () : 0);
+
+  return TRACE_ERR (err);
+}
+
+
 /* Release the data object with the handle DH.  */
 void
 gpgme_data_release (gpgme_data_t dh)
index e3ca165..769c2c8 100644 (file)
@@ -1165,9 +1165,8 @@ gpgme_error_t gpgme_data_new_from_filepart (gpgme_data_t *r_dh,
                                            const char *fname, FILE *fp,
                                            @API__OFF_T@ offset, size_t length);
 
-/* Reset the read pointer in DH.  Deprecated, please use
-   gpgme_data_seek instead.  */
-gpgme_error_t gpgme_data_rewind (gpgme_data_t dh) _GPGME_DEPRECATED(0,4);
+/* Convenience function to do a gpgme_data_seek (dh, 0, SEEK_SET).  */
+gpgme_error_t gpgme_data_rewind (gpgme_data_t dh);
 
 
 \f