Re: The bootstrap of Emacs on Cygwin is broken [Is also: unicode branch is merged to trunk]

[Angelo Graziosi]

Eli Zaretskii ha scritto:
> > Date: Tue, 05 Feb 2008 11:12:13 +0100
> > From: Angelo Graziosi <address@hidden>
> > CC: address@hidden, address@hidden, address@hidden
> >
> > -#define STATIC_HEAP_SIZE       (12 * 1024 * 1024)
> > +#ifdef CYGWIN
> > +  #define STATIC_HEAP_SIZE     (32 * 1024 * 1024)
> > +#else
> > +  #define STATIC_HEAP_SIZE     (12 * 1024 * 1024)
> > +#endif
> >
> >   int debug_sheap = 0;
> > ========================================================
> >
> > (Eli, if you object that things as "#ifdef CYGWIN" should go to 
> > cygwin.h, observe the sheap.c yet contains an "#ifdef CYGWIN sbrk...")
>
> Actually, I wonder why do you need "#ifdef CYGWIN" at all: sheap.c is
> compiled _only_ for the Cygwin build (see src/Makefile.in).  It looks
> like you can change the above line unconditionally.

This explains why I have not found it compiled on linux (a search for 
*heap in the build log finds w32heap.c).

In any case, if sheap.c is used only on Cygwin, this section of code

void *
bss_sbrk (ptrdiff_t request_size)
{
  if (!bss_sbrk_ptr)
    {
      bss_sbrk_ptr = bss_sbrk_buffer;
#ifdef CYGWIN
      sbrk (BLOCKSIZE);         /* force space for fork to work */
#endif
    }

  if (!(int) request_size)


should be changed too! or not?


Cheers,
   Angelo.