OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-226-g3e9d41c

Gerrit Code Review gerrit@openafs.org
Thu, 1 Dec 2022 11:37:15 -0500


The following commit has been merged in the master branch:
commit 3e9d41c98ecf07519ff6d2a1413615547ace13b8
Author: Cheyenne Wills <cwills@sinenomine.net>
Date:   Wed Nov 2 14:53:21 2022 -0600

    dumptool: Remove unused 'sflag' variable
    
    clang-16 is flagging unused-but-set variables which result in build
    errors when --enable-warning is turned on.
    
       ./dumptool.c:1285:54: error: variable 'sflag' set but not used
          [-Werror,-Wunused-but-set-variable]
           int errflg = 0, lflag = 0, iflag = 0, Fflag = 0, sflag = 0, Rflag = 0;
    
    Within dumptool the '-s' option of the 'ls' subcommand sets the 'sflag'
    variable, but the variable itself is never used anywhere else.
    
    Remove the variable 'sflag', and ignore the '-s' option.
    
    The sflag variable was unused in the commit that introduced the code:
    "test-suite-pull-tools-directly-in-20020114" (fcb32c07ed6d)
    
    Change-Id: I0769b9d3f76b4a802109628496bf332618162f89
    Reviewed-on: https://gerrit.openafs.org/15181
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/tools/dumpscan/dumptool.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository