mirror of
https://github.com/git/git.git
synced 2026-01-08 09:13:48 +00:00
config: use git_parse_int() in git_config_get_expiry_in_days()
git_config_get_expiry_in_days() calls git_parse_signed() with the maximum value of int, which is equivalent to calling git_parse_int(). Do that instead, as its shorter and clearer. This requires demoting "days" to int to match. Promote "scale" to intmax_t in turn to arrive at the same result when multiplying them. Signed-off-by: René Scharfe <l.s.r@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
committed by
Junio C Hamano
parent
9a2fb147f2
commit
06188ea5f3
6
config.c
6
config.c
@@ -2434,14 +2434,14 @@ int repo_config_get_expiry_in_days(struct repository *r, const char *key,
|
||||
timestamp_t *expiry, timestamp_t now)
|
||||
{
|
||||
const char *expiry_string;
|
||||
intmax_t days;
|
||||
int days;
|
||||
timestamp_t when;
|
||||
|
||||
if (repo_config_get_string_tmp(r, key, &expiry_string))
|
||||
return 1; /* no such thing */
|
||||
|
||||
if (git_parse_signed(expiry_string, &days, maximum_signed_value_of_type(int))) {
|
||||
const int scale = 86400;
|
||||
if (git_parse_int(expiry_string, &days)) {
|
||||
const intmax_t scale = 86400;
|
||||
*expiry = now - days * scale;
|
||||
return 0;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user