diff options
| author | Paul Eggert | 2015-11-19 11:49:27 -0800 |
|---|---|---|
| committer | Paul Eggert | 2015-11-19 11:50:16 -0800 |
| commit | 62d787ed124726fd1f73dcfb7227d83bc5940fcc (patch) | |
| tree | f2fbfa26961709470101b1898cef1763100f0a8d /src/module.c | |
| parent | c8a972b0c3082edfcca4a85562224499f75bfe9b (diff) | |
| download | emacs-62d787ed124726fd1f73dcfb7227d83bc5940fcc.tar.gz emacs-62d787ed124726fd1f73dcfb7227d83bc5940fcc.zip | |
Fix minor module problems found by static checking
* src/dynlib.c (dynlib_close): #ifdef out for now, as it’s not used.
* src/eval.c, src/lisp.h (lisp_eval_depth): Now static.
* src/module.c (Fmodule_load): Fix pointer signedness bug.
(Fmodule_call): Tell GCC that the default case is unreachable.
Diffstat (limited to 'src/module.c')
| -rw-r--r-- | src/module.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/module.c b/src/module.c index 35a9f48eb12..98264df35a9 100644 --- a/src/module.c +++ b/src/module.c | |||
| @@ -697,7 +697,7 @@ DEFUN ("module-load", Fmodule_load, Smodule_load, 1, 1, 0, | |||
| 697 | void *gpl_sym; | 697 | void *gpl_sym; |
| 698 | 698 | ||
| 699 | CHECK_STRING (file); | 699 | CHECK_STRING (file); |
| 700 | handle = dynlib_open (SDATA (file)); | 700 | handle = dynlib_open (SSDATA (file)); |
| 701 | if (!handle) | 701 | if (!handle) |
| 702 | error ("Cannot load file %s: %s", SDATA (file), dynlib_error ()); | 702 | error ("Cannot load file %s: %s", SDATA (file), dynlib_error ()); |
| 703 | 703 | ||
| @@ -789,6 +789,8 @@ ARGLIST is a list of arguments passed to SUBRPTR. */) | |||
| 789 | finalize_environment (&env); | 789 | finalize_environment (&env); |
| 790 | Fthrow (tag, value); | 790 | Fthrow (tag, value); |
| 791 | } | 791 | } |
| 792 | default: | ||
| 793 | eassume (false); | ||
| 792 | } | 794 | } |
| 793 | } | 795 | } |
| 794 | 796 | ||