public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* Patch ld saved_script_handle
@ 2001-08-15  8:14 Tom Rix
  2001-08-15  9:51 ` Nick Clifton
  0 siblings, 1 reply; 2+ messages in thread
From: Tom Rix @ 2001-08-15  8:14 UTC (permalink / raw)
  To: binutils

ld does not build (at least on aix 4.3.3) 

Traced the problem to initializing the variable saved_script_handle in ldgram.y
to false.  This patch initializes it to NULL.

Tom
-- 
Tom Rix 
GCC Engineer
trix@redhat.com
256.704.9201
2001-08-15  Tom Rix <trix@redhat.com>

	* ldgram.y (saved_script_handle): Initialize to NULL.
	* ldmain.c (main): Change check on saved_script_handle.

diff -rcp src-original/ld/ldgram.y src/ld/ldgram.y
*** src-original/ld/ldgram.y	Wed Aug 15 07:41:02 2001
--- src/ld/ldgram.y	Wed Aug 15 07:41:59 2001
*************** static enum section_type sectype;
*** 50,56 ****
  lang_memory_region_type *region;
  
  boolean ldgram_want_filename = true;
! FILE *  saved_script_handle = false;
  boolean force_make_executable = false;
  
  boolean ldgram_in_script = false;
--- 50,56 ----
  lang_memory_region_type *region;
  
  boolean ldgram_want_filename = true;
! FILE *  saved_script_handle = NULL;
  boolean force_make_executable = false;
  
  boolean ldgram_in_script = false;
diff -rcp src-original/ld/ldmain.c src/ld/ldmain.c
*** src-original/ld/ldmain.c	Wed Aug 15 07:41:02 2001
--- src/ld/ldmain.c	Wed Aug 15 07:43:37 2001
*************** main (argc, argv)
*** 298,304 ****
  
    /* If we have not already opened and parsed a linker script
       read the emulation's appropriate default script.  */
!   if (saved_script_handle == false)
      {
        int isfile;
        char *s = ldemul_get_script (& isfile);
--- 298,304 ----
  
    /* If we have not already opened and parsed a linker script
       read the emulation's appropriate default script.  */
!   if (saved_script_handle == NULL)
      {
        int isfile;
        char *s = ldemul_get_script (& isfile);

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

* Re: Patch ld saved_script_handle
  2001-08-15  8:14 Patch ld saved_script_handle Tom Rix
@ 2001-08-15  9:51 ` Nick Clifton
  0 siblings, 0 replies; 2+ messages in thread
From: Nick Clifton @ 2001-08-15  9:51 UTC (permalink / raw)
  To: Tom Rix; +Cc: binutils

Hi Tom,

> ld does not build (at least on aix 4.3.3) 
> 
> Traced the problem to initializing the variable saved_script_handle in ldgram.y
> to false.  This patch initializes it to NULL.

Doh!  My fault.  Thanks for catching this.

I assume that you have appleid the patch, but if not please do so.

Cheers
        Nick

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

end of thread, other threads:[~2001-08-15  9:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-08-15  8:14 Patch ld saved_script_handle Tom Rix
2001-08-15  9:51 ` Nick Clifton

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).