Hi Ben, > The prompt parameter to getpass() is declared as nonnull (using a GCC > nonnull attribute), but the implementation checks whether it is null in > two places. GCC warns about this. This commit removes the checks
GCC warnings ought to help us make the code more robust. Removing the NULL check makes it less robust. The problem has already occurred a couple of times: https://lists.gnu.org/archive/html/bug-gnulib/2020-01/msg00050.html https://lists.gnu.org/archive/html/bug-gnulib/2018-08/msg00116.html https://lists.gnu.org/archive/html/bug-gnulib/2013-02/msg00060.html https://lists.gnu.org/archive/html/bug-gnulib/2009-12/msg00173.html I would prefer that the same idiom gets used, that gets rid of the warning without removing the NULL check at run time. Bruno
