Skip to content

Commit 5818860

Browse files
committed
parse-options: adjust parse_opt_unknown_cb()s declared return type
In f41179f (parse-options: avoid magic return codes, 2019-01-27), the signature of the low-level parse-opt callback function was changed to return an `enum`. And while the implementations were changed, one declaration was left unchanged, still claiming to return `int`. This can potentially lead to problems, as compilers are free to choose any integral type for an `enum` as long as it can represent all declared values. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 7dbb037 commit 5818860

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

parse-options.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,9 @@ int parse_opt_commit(const struct option *, const char *, int);
293293
int parse_opt_tertiary(const struct option *, const char *, int);
294294
int parse_opt_string_list(const struct option *, const char *, int);
295295
int parse_opt_noop_cb(const struct option *, const char *, int);
296-
int parse_opt_unknown_cb(struct parse_opt_ctx_t *ctx, const struct option *, const char *, int);
296+
enum parse_opt_result parse_opt_unknown_cb(struct parse_opt_ctx_t *ctx,
297+
const struct option *,
298+
const char *, int);
297299
int parse_opt_passthru(const struct option *, const char *, int);
298300
int parse_opt_passthru_argv(const struct option *, const char *, int);
299301

0 commit comments

Comments
 (0)