Lose useless rpmgi reference counting
authorPanu Matilainen <pmatilai@redhat.com>
Fri, 19 Mar 2010 14:51:32 +0000 (16:51 +0200)
committerPanu Matilainen <pmatilai@redhat.com>
Fri, 19 Mar 2010 14:51:32 +0000 (16:51 +0200)
- these dont get passed around in a manner where refcounting would
  make any difference

lib/rpmgi.c
lib/rpmgi.h

index fdcbdec..1b614bf 100644 (file)
@@ -51,8 +51,6 @@ struct rpmgi_s {
     int ftsOpts;
     FTS * ftsp;
     FTSENT * fts;
-
-    int nrefs;                 /*!< Reference count. */
 };
 
 static const char * const ftsInfoStrings[] = {
@@ -362,39 +360,11 @@ fprintf(stderr, "\tav %p[%ld]: \"%s\" -> %s ~= \"%s\"\n", gi->argv, (long) (av -
     return rpmrc;
 }
 
-rpmgi rpmgiUnlink(rpmgi gi, const char * msg)
-{
-    if (gi == NULL) return NULL;
-
-if (_rpmgi_debug && msg != NULL)
-fprintf(stderr, "--> gi %p -- %d: %s\n", gi, gi->nrefs, msg);
-
-    gi->nrefs--;
-    return NULL;
-}
-
-rpmgi rpmgiLink(rpmgi gi, const char * msg)
-{
-    if (gi == NULL) return NULL;
-    gi->nrefs++;
-
-if (_rpmgi_debug && msg != NULL)
-fprintf(stderr, "--> gi %p ++ %d: %s\n", gi, gi->nrefs, msg);
-
-    return gi;
-}
-
 rpmgi rpmgiFree(rpmgi gi)
 {
     if (gi == NULL)
        return NULL;
 
-    if (gi->nrefs > 1)
-       return rpmgiUnlink(gi, __FUNCTION__);
-
-    (void) rpmgiUnlink(gi, __FUNCTION__);
-
-
     gi->hdrPath = _free(gi->hdrPath);
     gi->h = headerFree(gi->h);
 
@@ -447,8 +417,6 @@ rpmgi rpmgiNew(rpmts ts, rpmTag tag, const void * keyp, size_t keylen)
     gi->ftsp = NULL;
     gi->fts = NULL;
 
-    gi = rpmgiLink(gi, __FUNCTION__);
-
     return gi;
 }
 
index f005342..4fcf8f8 100644 (file)
@@ -39,22 +39,6 @@ typedef enum rpmgiFlags_e {
 
 extern rpmgiFlags giFlags;
 
-/** \ingroup rpmgi
- * Unreference a generalized iterator instance.
- * @param gi           generalized iterator
- * @param msg
- * @return             NULL always
- */
-rpmgi rpmgiUnlink (rpmgi gi, const char * msg);
-
-/** \ingroup rpmgi
- * Reference a generalized iterator instance.
- * @param gi           generalized iterator
- * @param msg
- * @return             new generalized iterator reference
- */
-rpmgi rpmgiLink (rpmgi gi, const char * msg);
-
 /** \ingroup rpmgi 
  * Destroy a generalized iterator.
  * @param gi           generalized iterator