From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: Eric Sandeen <sandeen@redhat.com>
Cc: linux-xfs <linux-xfs@vger.kernel.org>
Subject: Re: [PATCH] xfsprogs: remove unused function prototypes
Date: Mon, 8 Apr 2019 15:42:14 -0700 [thread overview]
Message-ID: <20190408224214.GC5147@magnolia> (raw)
In-Reply-To: <d708b5f4-a955-43eb-dbee-56e8a0606aa4@redhat.com>
On Mon, Apr 08, 2019 at 05:39:31PM -0500, Eric Sandeen wrote:
> These are prototypes for functions that aren't defined anywhere in
> the code, so remove them.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
LGTM
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
--D
> ---
>
> diff --git a/db/crc.h b/db/crc.h
> index ca2621f..e5cc53d 100644
> --- a/db/crc.h
> +++ b/db/crc.h
> @@ -7,4 +7,3 @@
> struct field;
>
> extern void crc_init(void);
> -extern void crc_struct(const field_t *fields, int argc, char **argv);
> diff --git a/db/faddr.h b/db/faddr.h
> index 371bef4..50c4b01 100644
> --- a/db/faddr.h
> +++ b/db/faddr.h
> @@ -13,7 +13,6 @@ extern void fa_cfileoffd(void *obj, int bit, typnm_t next);
> extern void fa_cfsblock(void *obj, int bit, typnm_t next);
> extern void fa_dfiloffd(void *obj, int bit, typnm_t next);
> extern void fa_dfsbno(void *obj, int bit, typnm_t next);
> -extern void fa_dinode_union(void *obj, int bit, typnm_t next);
> extern void fa_dirblock(void *obj, int bit, typnm_t next);
> extern void fa_drfsbno(void *obj, int bit, typnm_t next);
> extern void fa_drtbno(void *obj, int bit, typnm_t next);
> diff --git a/db/io.h b/db/io.h
> index 7618be4..c29a748 100644
> --- a/db/io.h
> +++ b/db/io.h
> @@ -46,8 +46,7 @@ extern void pop_cur(void);
> extern void print_iocur(char *tag, iocur_t *ioc);
> extern void push_cur(void);
> extern void push_cur_and_set_type(void);
> -extern int read_buf(int64_t daddr, int count, void *bufp);
> -extern void write_cur(void);
> +extern void write_cur(void);
> extern void set_cur(const struct typ *type, xfs_daddr_t blknum,
> int len, int ring_add, bbmap_t *bbmap);
> extern void ring_add(void);
> diff --git a/include/input.h b/include/input.h
> index 4878d69..6d8dbcc 100644
> --- a/include/input.h
> +++ b/include/input.h
> @@ -19,7 +19,6 @@ extern char *fetchline(void);
>
> extern size_t numlen(uint64_t val, size_t base);
>
> -extern struct timeval tadd(struct timeval t1, struct timeval t2);
> extern struct timeval tsub(struct timeval t1, struct timeval t2);
> extern double tdiv(double value, struct timeval tv);
>
> diff --git a/include/libxlog.h b/include/libxlog.h
> index 6c79115..92c54f4 100644
> --- a/include/libxlog.h
> +++ b/include/libxlog.h
> @@ -88,7 +88,6 @@ extern int xlog_find_cycle_start(struct xlog *log, xfs_buf_t *bp,
> extern int xlog_find_tail(struct xlog *log, xfs_daddr_t *head_blk,
> xfs_daddr_t *tail_blk);
>
> -extern int xlog_test_footer(struct xlog *log);
> extern int xlog_recover(struct xlog *log, int readonly);
> extern void xlog_recover_print_data(char *p, int len);
> extern void xlog_recover_print_logitem(xlog_recover_item_t *item);
> diff --git a/libxfs/libxfs_io.h b/libxfs/libxfs_io.h
> index 12064d7..5bcf733 100644
> --- a/libxfs/libxfs_io.h
> +++ b/libxfs/libxfs_io.h
> @@ -176,7 +176,6 @@ extern void libxfs_bcache_free(void);
> extern void libxfs_bcache_flush(void);
> extern void libxfs_purgebuf(xfs_buf_t *);
> extern int libxfs_bcache_overflowed(void);
> -extern int libxfs_bcache_usage(void);
>
> /* Buffer (Raw) Interfaces */
> extern xfs_buf_t *libxfs_getbufr(struct xfs_buftarg *, xfs_daddr_t, int);
> diff --git a/libxfs/xfs_dir2_priv.h b/libxfs/xfs_dir2_priv.h
> index 59f9fb2..08643a3 100644
> --- a/libxfs/xfs_dir2_priv.h
> +++ b/libxfs/xfs_dir2_priv.h
> @@ -119,8 +119,4 @@ extern int xfs_dir2_sf_removename(struct xfs_da_args *args);
> extern int xfs_dir2_sf_replace(struct xfs_da_args *args);
> extern xfs_failaddr_t xfs_dir2_sf_verify(struct xfs_inode *ip);
>
> -/* xfs_dir2_readdir.c */
> -extern int xfs_readdir(struct xfs_trans *tp, struct xfs_inode *dp,
> - struct dir_context *ctx, size_t bufsize);
> -
> #endif /* __XFS_DIR2_PRIV_H__ */
>
prev parent reply other threads:[~2019-04-08 22:42 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-08 22:39 [PATCH] xfsprogs: remove unused function prototypes Eric Sandeen
2019-04-08 22:42 ` Darrick J. Wong [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190408224214.GC5147@magnolia \
--to=darrick.wong@oracle.com \
--cc=linux-xfs@vger.kernel.org \
--cc=sandeen@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).