diff options
| author | Paul Eggert | 2016-12-06 15:25:54 -0800 |
|---|---|---|
| committer | Paul Eggert | 2016-12-06 15:26:11 -0800 |
| commit | c95270ad4b48204880ae10ec730eb121ffa14abe (patch) | |
| tree | 1f14c9708a157112ed4cc2c176d7fe5233c4819b /src/data.c | |
| parent | 38d0276ad122d1a7663ecca965506f85b4e29b7f (diff) | |
| download | emacs-c95270ad4b48204880ae10ec730eb121ffa14abe.tar.gz emacs-c95270ad4b48204880ae10ec730eb121ffa14abe.zip | |
Change two _Noreturn functions to return void
This is a bit clearer than _Noreturn functions that (do not)
return a non-void type.
* src/callproc.c (call_process) [MSDOS]:
Use 'status' local to record status.
(child_setup): Return CHILD_SETUP_TYPE.
* src/data.c, src/lisp.h (wrong_type_argument): Return void.
All callers changed.
* src/lisp.h (CHILD_SETUP_TYPE): New macro.
Diffstat (limited to 'src/data.c')
| -rw-r--r-- | src/data.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/data.c b/src/data.c index eee2a52a37a..64cd8b23b46 100644 --- a/src/data.c +++ b/src/data.c | |||
| @@ -138,7 +138,7 @@ wrong_length_argument (Lisp_Object a1, Lisp_Object a2, Lisp_Object a3) | |||
| 138 | make_number (bool_vector_size (a3))); | 138 | make_number (bool_vector_size (a3))); |
| 139 | } | 139 | } |
| 140 | 140 | ||
| 141 | Lisp_Object | 141 | _Noreturn void |
| 142 | wrong_type_argument (register Lisp_Object predicate, register Lisp_Object value) | 142 | wrong_type_argument (register Lisp_Object predicate, register Lisp_Object value) |
| 143 | { | 143 | { |
| 144 | /* If VALUE is not even a valid Lisp object, we'd want to abort here | 144 | /* If VALUE is not even a valid Lisp object, we'd want to abort here |
| @@ -2924,7 +2924,7 @@ float_arith_driver (double accum, ptrdiff_t argnum, enum arithop code, | |||
| 2924 | case Alogand: | 2924 | case Alogand: |
| 2925 | case Alogior: | 2925 | case Alogior: |
| 2926 | case Alogxor: | 2926 | case Alogxor: |
| 2927 | return wrong_type_argument (Qinteger_or_marker_p, val); | 2927 | wrong_type_argument (Qinteger_or_marker_p, val); |
| 2928 | case Amax: | 2928 | case Amax: |
| 2929 | if (!argnum || isnan (next) || next > accum) | 2929 | if (!argnum || isnan (next) || next > accum) |
| 2930 | accum = next; | 2930 | accum = next; |