unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Warning in lib/malloc.c
@ 2024-12-18 13:30 Eli Zaretskii
  2024-12-18 17:48 ` Paul Eggert
  0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2024-12-18 13:30 UTC (permalink / raw)
  To: Paul Eggert; +Cc: emacs-devel

Recent update from Gnulib causes the following compilation warning
when building with MinGW:

  malloc.c:32:1: warning: no previous prototype for 'rpl_malloc' [-Wmissing-prototypes]
     32 | rpl_malloc (size_t n)
	| ^~~~~~~~~~

I'm guessing that this is because nt/gnulib-cfg.mk does

  OMIT_GNULIB_MODULE_malloc-posix = true
  OMIT_GNULIB_MODULE_realloc-gnu = true
  OMIT_GNULIB_MODULE_realloc-posix = true

and thus does not include (and probably doesn't generate) some Gnulib
header.  I also think that the MinGW build should not compile
malloc.c, as I've seen no callers of rpl_malloc.  So maybe
nt/gnulib-cfg.mk should also do

  OMIT_GNULIB_MODULE_malloc-gnu = true

?  If you agree, I will make that change.

Thanks.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Warning in lib/malloc.c
  2024-12-18 13:30 Warning in lib/malloc.c Eli Zaretskii
@ 2024-12-18 17:48 ` Paul Eggert
  2024-12-18 19:17   ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Eggert @ 2024-12-18 17:48 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel

On 2024-12-18 05:30, Eli Zaretskii wrote:
> So maybe
> nt/gnulib-cfg.mk should also do
> 
>    OMIT_GNULIB_MODULE_malloc-gnu = true

Yes, that sounds right.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Warning in lib/malloc.c
  2024-12-18 17:48 ` Paul Eggert
@ 2024-12-18 19:17   ` Eli Zaretskii
  0 siblings, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2024-12-18 19:17 UTC (permalink / raw)
  To: Paul Eggert; +Cc: emacs-devel

> Date: Wed, 18 Dec 2024 09:48:20 -0800
> Cc: emacs-devel@gnu.org
> From: Paul Eggert <eggert@cs.ucla.edu>
> 
> On 2024-12-18 05:30, Eli Zaretskii wrote:
> > So maybe
> > nt/gnulib-cfg.mk should also do
> > 
> >    OMIT_GNULIB_MODULE_malloc-gnu = true
> 
> Yes, that sounds right.

Thanks.  It turned out to be more complicated than that, but now done.



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-12-18 19:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-18 13:30 Warning in lib/malloc.c Eli Zaretskii
2024-12-18 17:48 ` Paul Eggert
2024-12-18 19:17   ` Eli Zaretskii

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).