vol: Remove unused ihandle macros 61/14761/2
authorAndrew Deason <adeason@sinenomine.net>
Wed, 18 Aug 2021 23:29:09 +0000 (18:29 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Thu, 2 Sep 2021 22:38:16 +0000 (18:38 -0400)
All of these macros are not used anywhere in the tree; get rid of
them. Mostly this is just removing cruft, but removing
FDH_READ/FDH_WRITE/FDH_SEEK also helps make sure we don't accidentally
reintroduce code paths that use non-positional i/o.

No functional change should be incurred by this commit.

Change-Id: I9f6e885845e814cdb22d7c963e4ee6826cad8cea
Reviewed-on: https://gerrit.openafs.org/14761
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/vol/ihandle.h

index e79b4d1..c4d307b 100644 (file)
@@ -58,8 +58,6 @@
  * IH_INC/IH_DEC - increment/decrement the link count.
  *
  * Replacements for C runtime file operations
- * FDH_READ/FDH_WRITE - read/write using the file descriptor.
- * FDH_SEEK - set file handle's read/write position
  * FDH_CLOSE - return a file descriptor to the cache
  * FDH_REALLYCLOSE - Close a file descriptor, do not return to the cache
  * FDH_SYNC - Unconditionally sync an open file.
@@ -357,8 +355,6 @@ extern int fd_blocksize(FdHandle_t *fdP, afs_sfsize_t *a_size,
 
 #define IH_COPY(D, S) ((D) = ih_copy(S))
 
-#define IH_NLINK(H) ih_nlink(H)
-
 #define IH_OPEN(H) ih_open(H)
 
 #define FDH_CLOSE(H) (fd_close(H), (H)=NULL)
@@ -367,8 +363,6 @@ extern int fd_blocksize(FdHandle_t *fdP, afs_sfsize_t *a_size,
 
 #define FDH_FDOPEN(H, A) stream_fdopen((H)->fd_fd)
 
-#define STREAM_FDOPEN(A, B) stream_fdopen(A)
-
 #define STREAM_OPEN(A, B) stream_open(A, B)
 
 #define STREAM_READ(A, B, C, H) stream_read(A, B, C, H)
@@ -579,9 +573,6 @@ extern afs_sfsize_t ih_size(FD_t);
 
 #define FDH_PREAD(H, B, S, O) OS_PREAD((H)->fd_fd, B, S, O)
 #define FDH_PWRITE(H, B, S, O) OS_PWRITE((H)->fd_fd, B, S, O)
-#define FDH_READ(H, B, S) OS_READ((H)->fd_fd, B, S)
-#define FDH_WRITE(H, B, S) OS_WRITE((H)->fd_fd, B, S)
-#define FDH_SEEK(H, O, F) OS_SEEK((H)->fd_fd, O, F)
 
 #define FDH_SYNC(H) ih_fdsync(H)
 #define FDH_TRUNC(H, L) OS_TRUNC((H)->fd_fd, L)