Ignore pending_signals when checking for quits.

pending_signals is often set if no quit is pending.  This results in
bugs in module code if the module returns but no quit is actually

* src/emacs-module.c (module_should_quit): Use QUITP macro to check
whether the caller should quit.

* src/eval.c: Remove obsolete comment.
1 job for emacs-26 in 28 minutes and 4 seconds (queued for 2 seconds)
Status Job ID Name Coverage
failed #2924


Name Stage Failure
test Test
Makefile:285: recipe for target 'check-doit' failed
make[2]: *** [check-doit] Error 1
make[2]: Leaving directory '/builds/emacs/emacs/test'
Makefile:255: recipe for target 'check' failed
make[1]: *** [check] Error 2
make[1]: Leaving directory '/builds/emacs/emacs/test'
Makefile:951: recipe for target 'check' failed
make: *** [check] Error 2
ERROR: Job failed: exit code 1