Whamcloud - gitweb
lib/{e2p,ss}: remove manual declarations of errno
authorEric Biggers <ebiggers@google.com>
Sat, 21 Jan 2023 20:32:05 +0000 (12:32 -0800)
committerTheodore Ts'o <tytso@mit.edu>
Fri, 27 Jan 2023 17:33:59 +0000 (12:33 -0500)
commitdd54f64593b0a44e745af9934506cf0791d16ef3
tree30c9584cc17c15ce9d5ad1e540269042f8d383de
parentc0ffbcd4dbd64e795e8766b0bf152979ed29d69f
lib/{e2p,ss}: remove manual declarations of errno

As per 'man 3 errno':

    On some ancient systems, <errno.h> was not present or did not
    declare errno, so that it was necessary to declare errno manually
    (i.e., extern int errno).   **Do not do this**.  It long ago ceased
    to be necessary, and it will cause problems with modern versions of
    the C library.

One of the platforms it causes a problem on is Windows:

    In file included from fgetversion.c:28:
    fgetversion.c: In function ‘fgetversion’:
    fgetversion.c:68:20: warning: ‘_errno’ redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]
       68 |         extern int errno;
          |                    ^~~~~

Just remove these obsolete manual declarations of errno.

Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
lib/e2p/fgetversion.c
lib/e2p/fsetversion.c
lib/e2p/getversion.c
lib/e2p/setversion.c
lib/ss/execute_cmd.c
lib/ss/help.c
lib/ss/pager.c