]> Savannah Git Hosting - gnulib.git/commit
Move xstrtol_fatal to a new xstrtol-error module
authorPino Toscano <ptoscano@redhat.com>
Thu, 5 Dec 2019 15:18:35 +0000 (16:18 +0100)
committerBruno Haible <bruno@clisp.org>
Sat, 7 Dec 2019 09:39:56 +0000 (10:39 +0100)
commita762d02b8efb0754e5d9f4491588214130a2657f
treed0796c7078a2ee954231696a291f0fddb80c39c6
parent1c740444c9224c81b92c4375babc73efe57e1dc9
Move xstrtol_fatal to a new xstrtol-error module

The xstrtol module provides a xstrtol_fatal function which uses other
modules suitable mostly for command line handling (e.g. gettext,
getopt), and that are completely unused when using only xstrto*
functions. Furthermore, xstrtol_fatal is used only in the xstrtol-tests
(within gnulib itself).

As solution, move the xstrtol_fatal to a new xstrtol-error module,
making xstrtol-tests depend on it. Since the prototype of xstrtol_fatal
is automatically provided by xstrtol.h, it is very difficult to know
whether anyone is actually using it, so add a note about this in NEWS.

* lib/xstrtol.h: Stop including <getopt.h>.
(xstrtol_fatal): Move ...
* lib/xstrtol-error.h: ... here.  New file.
* lib/xstrtol-error.c: Include xstrtol-error.h instead of xstrtol.h.
* tests/test-xstrtol.c: Likewise.
* modules/xstrtol (Files): Remove lib/xstrtol-error.c.
(Depends-on): Remove exitfail, error, getopt-gnu, and gettext-h.
(Makefile.am): Remove xstrtol-error.c from lib_SOURCES.
* modules/xstrtol-error: New file.
* modules/xstrtol-tests (Depends-on): Add xstrtol-error.
* MODULES.html.sh: Add xstrtol-error.
* NEWS: Document the change.
ChangeLog
MODULES.html.sh
NEWS
lib/xstrtol-error.c
lib/xstrtol-error.h [new file with mode: 0644]
lib/xstrtol.h
modules/xstrtol
modules/xstrtol-error [new file with mode: 0644]
modules/xstrtol-tests
tests/test-xstrtol.c