diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2023-08-07 21:23:28 -0700 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2023-08-07 21:26:36 -0700 |
commit | 85b6c150c8a356ebf3442fde2836364954aa938c (patch) | |
tree | 0ea86860bb95571114c9a14a45a77b34f123bac6 /test/lisp/emacs-lisp/warnings-tests.el | |
parent | a579739e2b3f3c7218b14a7e41b78e7ddbded06c (diff) | |
download | emacs-85b6c150c8a356ebf3442fde2836364954aa938c.tar.gz emacs-85b6c150c8a356ebf3442fde2836364954aa938c.tar.bz2 emacs-85b6c150c8a356ebf3442fde2836364954aa938c.zip |
Fix some emacs_fopen confusion
Problem reported by Po Lu in:
https://lists.gnu.org/r/emacs-devel/2023-08/msg00195.html
* src/comp.c (comp_hash_source_file, Fcomp__release_ctxt):
* src/sysdep.c (get_up_time, procfs_ttyname, procfs_get_total_memory):
Be more systematic about using emacs_fclose on streams that were
opened with emacs_fopen or emacs_fdopen. Do this even if not
Android, as this simplifies checking that it's done consistently.
* src/lisp.h (emacs_fclose): If it’s just fclose,
make it a macro rather than a function, to avoid confusing gcc
-Wmismatched-dealloc.
(emacs_fopen): Move decl here from sysstdio.h, because sysstdio.h
is included from non-Emacs executables and emacs_fopen is good
only inside Emacs.
* src/sysdep.c (emacs_fclose): Define as a function only if Android.
Diffstat (limited to 'test/lisp/emacs-lisp/warnings-tests.el')
0 files changed, 0 insertions, 0 deletions