On Mon, Jan 27, 2020 at 04:56:02PM -0600, Eric Sandeen wrote:
> From: Eric Sandeen <sandeen@xxxxxxxxxx>
>
> In each of these cases, db, logprint, and mdrestore are redeclaring
> as a global variable something which was already provided by a
> library they link with.
Er... which library?
Also, uh...maybe we shouldn't be exporting globals across libraries?
(He says having not looked for how many there are lurki... ye gods)
--D
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> ---
>
> diff --git a/db/init.c b/db/init.c
> index 455220a..0ac3736 100644
> --- a/db/init.c
> +++ b/db/init.c
> @@ -27,7 +27,6 @@ static int force;
> static struct xfs_mount xmount;
> struct xfs_mount *mp;
> static struct xlog xlog;
> -libxfs_init_t x;
> xfs_agnumber_t cur_agno = NULLAGNUMBER;
>
> static void
> diff --git a/logprint/logprint.c b/logprint/logprint.c
> index 7754a2a..511a32a 100644
> --- a/logprint/logprint.c
> +++ b/logprint/logprint.c
> @@ -24,7 +24,6 @@ int print_buffer;
> int print_overwrite;
> int print_no_data;
> int print_no_print;
> -int print_exit = 1; /* -e is now default. specify -c to override */
> static int print_operation = OP_PRINT;
>
> static void
> @@ -132,6 +131,7 @@ main(int argc, char **argv)
> bindtextdomain(PACKAGE, LOCALEDIR);
> textdomain(PACKAGE);
> memset(&mount, 0, sizeof(mount));
> + print_exit = 1; /* -e is now default. specify -c to override */
>
> progname = basename(argv[0]);
> while ((c = getopt(argc, argv, "bC:cdefl:iqnors:tDVv")) != EOF) {
> @@ -152,7 +152,7 @@ main(int argc, char **argv)
> case 'e':
> /* -e is now default
> */
> - print_exit++;
> + print_exit = 1;
> break;
> case 'C':
> print_operation = OP_COPY;
> diff --git a/mdrestore/xfs_mdrestore.c b/mdrestore/xfs_mdrestore.c
> index 3375e08..1cd399d 100644
> --- a/mdrestore/xfs_mdrestore.c
> +++ b/mdrestore/xfs_mdrestore.c
> @@ -7,7 +7,6 @@
> #include "libxfs.h"
> #include "xfs_metadump.h"
>
> -char *progname;
> static int show_progress = 0;
> static int show_info = 0;
> static int progress_since_warning = 0;
>
