gbranden pushed a commit to branch master
in repository groff.
commit 52254689bc991cdf2f771740395e9035700cb36a
Author: Alejandro Colomar <[email protected]>
AuthorDate: Sat Mar 16 13:34:38 2024 +0100
[libgroff]: Remove dead code from current_time().
* src/libs/libgroff/curtime.cpp (current_time): Remove dead code.
strtol(3) can only report ERANGE, if the base is valid (and it is).
Fixes: e4290210f2c1 ("Implement `SOURCE_DATE_EPOCH' for reproducible
builds.")
Link: <https://savannah.gnu.org/bugs/?65451>
Cc: "G. Branden Robinson" <[email protected]>
Cc: Dave Kemper <[email protected]>
Cc: "James K. Lowden" <[email protected]>
Cc: Colin Watson <[email protected]>
Cc: Werner LEMBERG <[email protected]>
Cc: James Clark <[email protected]>
Signed-off-by: Alejandro Colomar <[email protected]>
---
ChangeLog | 6 ++++++
src/libs/libgroff/curtime.cpp | 2 +-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index df2566703..68ed9d28a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2024-03-16 Alejandro Colomar <[email protected]>
+
+ * src/libs/libgroff/curtime.cpp (current_time): Remove dead
+ code. strtol(3) can only report ERANGE, if the base is valid
+ {and it is}.
+
2024-03-16 Alejandro Colomar <[email protected]>
* src/libs/libgroff/curtime.cpp (current_time): Remove redundant
diff --git a/src/libs/libgroff/curtime.cpp b/src/libs/libgroff/curtime.cpp
index 490fc3018..4623ad9a0 100644
--- a/src/libs/libgroff/curtime.cpp
+++ b/src/libs/libgroff/curtime.cpp
@@ -42,7 +42,7 @@ struct tm *current_time()
char *endptr;
long epoch = strtol(source_date_epoch, &endptr, 10);
- if ((errno == ERANGE) || (errno != 0 && epoch == 0))
+ if (errno == ERANGE)
fatal("$SOURCE_DATE_EPOCH: strtol: %1", strerror(errno));
if (endptr == source_date_epoch)
fatal("$SOURCE_DATE_EPOCH: no digits found: '%1'", endptr);
_______________________________________________
Groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit
