(decode_field_spec): Add an abort after

`error (EXIT_FAILURE, ...' to avoid a gcc warning in caller,
about variables being used uninitialized.

1 files changed, 6 insertions, 0 deletions

diff --git a/src/join.c b/src/join.c
index a1c1f4ada..8f4461206 100644
--- a/src/join.c
+++ b/src/join.c
@@ -664,6 +664,12 @@ decode_field_spec (const char *s, int *file_index, size_t *field_index)
     default:
       error (EXIT_FAILURE, 0,
 	     _("invalid file number in field spec: %s"), quote (s));
+
+      /* Tell gcc -W -Wall that we can't get beyond this point.
+	 This avoids a warning (otherwise legit) that the caller's copies
+	 of *file_index and *field_index might be used uninitialized.  */
+      abort ();
+
       break;
     }
 }