Move pile of helpers from rpmio_internal.h to rpmio.c, remove unused stuff
authorPanu Matilainen <pmatilai@redhat.com>
Wed, 14 Oct 2009 13:10:08 +0000 (16:10 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Wed, 14 Oct 2009 13:10:08 +0000 (16:10 +0300)
- nothing outside rpmio.c needs these, make 'em static there
- uninlining reveals quite a few completely unused helpers...

rpmio/rpmio.c
rpmio/rpmio_internal.h

index 9ad1547..224842b 100644 (file)
@@ -29,6 +29,64 @@ extern int h_errno;
 
 #include "debug.h"
 
+static FDIO_t fdGetIo(FD_t fd)
+{
+    FDSANE(fd);
+    return fd->fps[fd->nfps].io;
+}
+
+static void fdSetIo(FD_t fd, FDIO_t io)
+{
+    FDSANE(fd);
+    fd->fps[fd->nfps].io = io;
+}
+
+static void * fdGetFp(FD_t fd)
+{
+    FDSANE(fd);
+    return fd->fps[fd->nfps].fp;
+}
+
+static void fdSetFp(FD_t fd, void * fp)
+{
+    FDSANE(fd);
+    fd->fps[fd->nfps].fp = fp;
+}
+
+static void fdSetFdno(FD_t fd, int fdno)
+{
+    FDSANE(fd);
+    fd->fps[fd->nfps].fdno = fdno;
+}
+
+static void fdPush(FD_t fd, FDIO_t io, void * fp, int fdno)
+{
+    FDSANE(fd);
+    if (fd->nfps >= (sizeof(fd->fps)/sizeof(fd->fps[0]) - 1))
+       return;
+    fd->nfps++;
+    fdSetIo(fd, io);
+    fdSetFp(fd, fp);
+    fdSetFdno(fd, fdno);
+}
+
+static void fdPop(FD_t fd)
+{
+    FDSANE(fd);
+    if (fd->nfps < 0) return;
+    fdSetIo(fd, NULL);
+    fdSetFp(fd, NULL);
+    fdSetFdno(fd, -1);
+    fd->nfps--;
+}
+
+static FD_t c2f(void * cookie)
+{
+    FD_t fd = (FD_t) cookie;
+    FDSANE(fd);
+    return fd;
+}
+
 #define FDNREFS(fd)    (fd ? ((FD_t)fd)->nrefs : -9)
 #define FDTO(fd)       (fd ? ((FD_t)fd)->rd_timeoutsecs : -99)
 #define FDCPIOPOS(fd)  (fd ? ((FD_t)fd)->fd_cpioPos : -99)
@@ -647,15 +705,6 @@ static ssize_t ufdWrite(void * cookie, const char * buf, size_t count)
     int bytesWritten;
     int total = 0;
 
-#ifdef NOTYET
-    if (fdGetIo(fd) == fdio) {
-       struct stat sb;
-       (void) fstat(fdGetFdno(fd), &sb);
-       if (S_ISREG(sb.st_mode))
-           return fdWrite(fd, buf, count);
-    }
-#endif
-
     UFDONLY(fd);
 
     for (total = 0; total < count; total += bytesWritten) {
index 9e5bc26..acca88e 100644 (file)
@@ -73,109 +73,6 @@ extern "C" {
 /** \ingroup rpmio
  */
 static inline
-FDIO_t fdGetIo(FD_t fd)
-{
-    FDSANE(fd);
-    return fd->fps[fd->nfps].io;
-}
-
-/** \ingroup rpmio
- */
-/* FIX: io may be NULL */
-static inline
-void fdSetIo(FD_t fd, FDIO_t io)
-{
-    FDSANE(fd);
-    fd->fps[fd->nfps].io = io;
-}
-
-/** \ingroup rpmio
- */
-static inline
-void * fdGetFp(FD_t fd)
-{
-    FDSANE(fd);
-    return fd->fps[fd->nfps].fp;
-}
-
-/** \ingroup rpmio
- */
-/* FIX: fp may be NULL */
-static inline
-void fdSetFp(FD_t fd, void * fp)
-{
-    FDSANE(fd);
-    fd->fps[fd->nfps].fp = fp;
-}
-
-/** \ingroup rpmio
- */
-static inline
-int fdGetFdno(FD_t fd)
-{
-    FDSANE(fd);
-    return fd->fps[fd->nfps].fdno;
-}
-
-/** \ingroup rpmio
- */
-static inline
-void fdSetFdno(FD_t fd, int fdno)
-{
-    FDSANE(fd);
-    fd->fps[fd->nfps].fdno = fdno;
-}
-
-/** \ingroup rpmio
- */
-static inline
-void fdPush(FD_t fd, FDIO_t io, void * fp, int fdno)
-{
-    FDSANE(fd);
-    if (fd->nfps >= (sizeof(fd->fps)/sizeof(fd->fps[0]) - 1))
-       return;
-    fd->nfps++;
-    fdSetIo(fd, io);
-    fdSetFp(fd, fp);
-    fdSetFdno(fd, fdno);
-}
-
-/** \ingroup rpmio
- */
-static inline
-void fdPop(FD_t fd)
-{
-    FDSANE(fd);
-    if (fd->nfps < 0) return;
-    fdSetIo(fd, NULL);
-    fdSetFp(fd, NULL);
-    fdSetFdno(fd, -1);
-    fd->nfps--;
-}
-
-
-/** \ingroup rpmio
- */
-static inline
-void fdSetSyserrno(FD_t fd, int syserrno, const void * errcookie)
-{
-    FDSANE(fd);
-    fd->syserrno = syserrno;
-    fd->errcookie = errcookie;
-}
-
-/** \ingroup rpmio
- */
-static inline
-int fdGetRdTimeoutSecs(FD_t fd)
-{
-    FDSANE(fd);
-    return fd->rd_timeoutsecs;
-}
-
-/** \ingroup rpmio
- */
-static inline
 rpm_loff_t fdGetCpioPos(FD_t fd)
 {
     FDSANE(fd);
@@ -199,16 +96,6 @@ void fdSetBundle(FD_t fd, rpmDigestBundle bundle)
 }
 
 /** \ingroup rpmio
- */
-static inline
-FD_t c2f(void * cookie)
-{
-    FD_t fd = (FD_t) cookie;
-    FDSANE(fd);
-    return fd;
-}
-
-/** \ingroup rpmio
  * Attach digest to fd.
  */
 void fdInitDigest(FD_t fd, pgpHashAlgo hashalgo, int flags);