(elf_link_input_bfd): Handle the case where a badly formatted input file results
in a reloc which has no associated global symbol

* link.cmd: Remove.

* syslex.l: Adjust top-of-file comment: this file is part of GNU
binutils, not GNU ld.

*** empty log message ***

* configure.tgt (*-*-freebsd*): Set gdb_osabi to
GDB_OSABI_FREEBSD_ELF.  Partial fix for PR gdb/1922.

* ld-fastcall/fastcall.exp: Don't run on i*86-*-openbsd*.

* ld-srec/srec.exp (run_srec_test): Deal with ProPolice on
*-*-openbsd*.

gas/

2005-04-26  H.J. Lu  <hongjiu.lu@intel.com>

* config/obj-multi.h (FAKE_LABEL_NAME): Defined.

* read.c (pseudo_set): Disallow symbol set to common symbol.

PR 857
* write.c (write_object_file): Report common symbol name when
disallowing local symbol set to common symbol.
(adjust_reloc_syms): Disallow local symbol set to undefined
symbol.

gas/testsuite/

2005-04-26  H.J. Lu  <hongjiu.lu@intel.com>

* gas/all/assign.s: Make `x' and `y' global.

Update BFD_VERSION_DATE to 20050426.

2005-04-26  Andrew Cagney  <cagney@gnu.org>

* remote.c (remote_open_1): Move "ex"'s declaration to where it is
used.
(remote_get_thread_local_address): Use throw_error, include a
printed string.
* linux-thread-db.c (thread_db_get_thread_local_address): Ditto.
* dwarf2loc.c (dwarf_expr_tls_address): Ditto.
* cli/cli-script.c (script_from_file): Mark up throw_error message.
* linespec.c (symtab_from_filename, decode_variable): Ditto.

* configure.in: Fix the check for basename declaration.
* configure: Regenerate.

* bfd.m4 (BFD_NEED_DECLARATION): Restore.

2005-04-26  Andrew Cagney  <cagney@gnu.org>

Rename 'struct exception' to 'struct gdb_exception'.
* wrapper.c: Update.
* varobj.c: Update.
* tui/tui-interp.c: Update.
* remote.c: Update.
* mi/mi-main.c: Update.
* mi/mi-interp.c: Update.
* linux-thread-db.c: Update.
* interps.h: Update.
* interps.c: Update.
* exceptions.h: Update.
* exceptions.c: Update.
* dwarf2loc.c: Update.
* cli/cli-interp.c: Update.
* cli/cli-script.c: Update.
* breakpoint.c: Update.

merge from gcc

*** empty log message ***

* event-loop.c (gdb_select): Program defensively.

* elfxx-sparc.c (sparc_elf_append_rela_64): Add BFD64 protection.
(sparc_elf_r_info_64, _bfd_sparc_elf_finish_dynamic_symbol,
sparc64_finish_dyn, _bfd_sparc_elf_finish_dynamic_sections):
Likewise.

* event-loop.c (gdb_assert.h): Include.
(<windows.h>): Include under Windows.
(<io.h>): Likeiwse.
(gdb_select): New function.
(gdb_wait_for_event): Use it.
* Makefile.in (event-loop.o): Depend on $(gdb_assert_h).

bfd/

2005-04-25  H.J. Lu  <hongjiu.lu@intel.com>

PR 858
* elflink.c (elf_link_input_bfd): Make non-debugging reference
to discarded section an error.

ld/testsuite/

2005-04-25  H.J. Lu  <hongjiu.lu@intel.com>

* ld-discard/extern.d: Expect error.
* ld-discard/start.d: Likewise.
* ld-discard/static.d: Likewise.

PR872
* objcopy.c (copy_archive): Initialise 'obfd' field of new name_list structure.

Oops - forgot one place where -K is documented.

* objcopy.c (copy_usage): Fix description of -K switch.
* doc/binutils.texi (strip, objcopy): Fix description of -K switch.

* corefile.h (core_num_syms): Remove unused extern declaration.
(core_syms): Likewise.
(min_insn_size): Likewise.

* armemu.c (handle_v6_insn): New function - emulate a few of the v6 instructions - the ones now generated by GCC.
  (ARMulEmulate32): Call handle_v6_insn when a possible v6 insn is found.
* armdefs.h (struct ARMul_State): Add new field: is_v6.#
  (ARM_v6_Prop): Define.
* arminit.c (ARMul_NewState): Initialise the v6 flag.
  (ARMul_SelectProcessor): Determine if the v6 flag should be set.
* wrapper.c (sim_create_inferior): For unknown architectures, default to allowing the v6 instructions.

gas/
2005-04-25  Jan Beulich  <jbeulich@novell.com>

* macro.c (macro_expand_body): Replace locals indicator parameters
with actual macro_entry. New local variables macro_line and err.
Don't return when encountering an error, just record the fact.
Detect local symbol name colliding with parameter. Track line number
inside of macro expansion.
(do_formals): Move local variable name to wider scope. Check
parameter of the same name doesn't already exist. In MRI mode, also
check it doesn't collide with the argument count pseudo-parameter).
(define_macro): Add file and line number parameters. Remove local
variable namestr. New local variable error. Initialize macro_entry
members file, line, and name. Don't return when encountering an
error, just record the fact. Use %s in some diagnostics for read.c
to insert the macro name. Free macro_entry on error.
(macro_expand): Pass macro_entry to macro_epand_body. Don't return
when encountering an error, just record the fact.
(expand_irp): Don't return when encountering an error, just record
the fact.
* macro.h (macro_struct): New members name, file, and line.
(define_macro): Add file and line number parameters.
* read.c (s_macro): Pass file and line to define_macro. Tag warning
regarding pseudo-op redefinition with the file/line that macro
definition started at.

gas/testsuite/
2005-04-25  Jan Beulich  <jbeulich@novell.com>

* gas/macros/badarg.s: Add tests for collisions between/among macro
parameters and local symbols.
* gas/macros/badarg.l: Adjust.

*** empty log message ***

* configure.host: Add *-*-openbsd support.

*** empty log message ***

* sparc64-tdep.c: Include "dwarf2-frame.h".
(sparc64_dwarf2_frame_init_reg): New function.
* Makefile.in (sparc64-tdep.o): Update dependencies.

* corefile.c (core_num_syms): Make static.
(core_syms): Likewise.
(min_insn_size): Likewise.
* gprof.c (abfd): Remove unused variable.
* hist.c (hist_dimension): Make static.
(hist_dimension_abbrev): Likewise.
* sym_ids.c (id_list): Likewise.
(table_name): Likewise.

* sparc64-tdep.c: Include "dwarf2-frame.h".
(sparc64_dwarf2_frame_init_reg): New function.
* Makefile.in (sparc64-tdep.o): Update dependencies.

*** empty log message ***

2005-04-22  David S. Miller  <davem@davemloft.net>

* sparc64-linux-tdep.c (sparc64_linux_init_abi): Add call to
sparc64_init_abi().

* config/tc-xtensa.c (LOOKAHEAD_ALIGNER): Delete macro.
(future_alignment_required): Remove ifdefs that use it.

* ser-base.c: Include "ser-base.h" instead of "ser-unix.h".

* ser-base.c (<sys/time.h>): Include.

* ppcobsd-tdep.c (ppcobsd_init_abi): Set return_value.

* config/tc-xtensa.c (xg_get_build_instr_size): Remove.
(xg_is_narrow_insn, xg_expand_narrow): Remove.  Merge into...
(xg_is_single_relaxable_insn): ...here.  Add "targ" and "narrow_only"
parameters.
(xg_assembly_relax, xg_find_narrowest_format, relaxation_requirements,
convert_frag_narrow): Use new version of xg_is_single_relaxable_insn.

* sparc-linux-tdep.c: Reorder includes.  Tweak a few comments.
(LINUX32_SIGTRAMP_INSN0, LINUX32_SIGTRAMP_INSN1)
(LINUX32_RT_SIGTRAMP_INSN0, LINUX32_RT_SIGTRAMP_INSN1): Remove
macros.
(sparc32_linux_sigframe, sparc32_linux_rt_sigframe): Hardcode
instructions.
* sparc64-linux-tdep.c: Reorder includes.  Tweak a few comments.
(LINUX64_RT_SIGTRAMP_INSN0, LINUX64_RT_SIGTRAMP_INSN1): Remove
macros.
(sparc64_linux_rt_sigframe): Hardcode instructions.
* Makefile.in (sparc-linux-tdep.o, sparc64-linux-tdep.o): Update

* sparc-linux-tdep.c: Reorder includes.  Tweak a few comments.
(LINUX32_SIGTRAMP_INSN0, LINUX32_SIGTRAMP_INSN1)
(LINUX32_RT_SIGTRAMP_INSN0, LINUX32_RT_SIGTRAMP_INSN1): Remove
macros.
(sparc32_linux_sigframe, sparc32_linux_rt_sigframe): Hardcode
instructions.
* sparc64-linux-tdep.c: Reorder includes.  Tweak a few comments.
(LINUX64_RT_SIGTRAMP_INSN0, LINUX64_RT_SIGTRAMP_INSN1): Remove
macros.
(sparc64_linux_rt_sigframe): Hardcode instructions.
* Makefile.in (sparc-linux-tdep.o, sparc64-linux-tdep.o): Update
dependencies.

* sparc-linux-tdep.c: Update copyright year.
(sparc32_linux_init_abi): Set skip_trampoline_code and
solib_svr4_fetch_link_map_offsets.  Set TDEP->plt_entry_size.
Don't set software_single_step and don't call
sparc32_sol2_init_abi.
* sparc64-linux-tdep.c: Update copyright year.
(sparc64_linux_init_abi): Set skip_trampoline_code and
solib_svr4_fetch_link_map_offsets.  Set TDEP->plt_entry_size.
Don't set software_single_step and don't call
sparc64_sol2_init_abi.

* configure.in: Replace BFD_NEED_DECLARATION checks by the
corresponding AC_CHECK_DECLS.
* sysdep.h: Replace NEED_DECLARATION_* checks by the corresponding
HAVE_DECL_*.
* bfd.m4 (BFD_NEED_DECLARATION): Remove, obsolete.
* configure: Regenerate.
* config.in: Ditto.

* gdb.texinfo (Remote configuration): Document "set/show
get-thread-local-storage-address".  Add cross-reference to the
description of the qGetTLSAddr packet.
(General Query Packets): Mention "set remote
get-thread-local-storage-address" and add a reference to its
description.

* remote-st.c (_initialize_remote_st2000): Remove the extraneous
"<command>" string from the call to add_com.
* remote-sim.c (_initialize_remote_sim): Ditto.
* remote-utils.c (_initialize_sr_support): Ditto.
* remote-mips.c (_initialize_remote_mips): Ditto.

2005-04-21  H.J. Lu  <hongjiu.lu@intel.com>

* Makefile.am (INCLUDES): New. Add -D_GNU_SOURCE.
(chew.o): Use it.
* Makefile.in: Regenerate.

*** empty log message ***

daily update

* sparc-linux-tdep.c (sparc32_linux_sigframe_init): New function.
(sparc32_linux_sigframe, sparc32_linux_rt_sigframe): New tramp_frames.
(sparc_linux_sigtramp_start, sparc32_linux_sigtramp_start,
sparc32_linux_rt_sigtramp_start, sparc32_linux_sigtramp_p,
sparc32_linux_sigtramp_frame_cache,
sparc32_linux_sigtramp_frame_this_id,
sparc32_linux_sigtramp_frame_prev_register,
sparc32_linux_sigtramp_frame_unwind,
sparc32_linux_sigtramp_frame_sniffer): Delete.
(sparc32_linux_init_abi): Register new tramp unwinders.  Kill
register of old sigtramp sniffer.
* sparc64-linux-tdep.c (sparc64_linux_sigframe_init): New
function.
(sparc64_linux_rt_sigframe): New tramp frame.
(sparc64_linux_sigtramp_start, sparc64_linux_sigtramp_p,
sparc64_linux_sigtramp_frame_cache,
sparc64_linux_sigtramp_frame_this_id,
sparc64_linux_sigtramp_frame_prev_register,
sparc64_linux_sigtramp_frame_unwind,
sparc64_linux_sigtramp_frame_sniffer): Delete.
(sparc64_linux_init_abi): Register new tramp unwinders.  Kill
register of old sigtramp sniffer.
Makefile.in (sparc64-linux-tdep.o): Update dependencies.
(sparc-linux-tdep.o): Likewise.

* ser-base.c: Include "gdb_string.h".
* Makefile.in (ser-base.o): Update dependencies.

* config/tc-z8k.c (md_assemble): Fix buffer overrun in operand[]
array.

* sparc-linux-tdep.c (sparc32_linux_sigtramp_p): Fix
signal return stub symbol names, __restore and __restore_rt are
the i386 symbol names not the sparc ones.
(sparc32_linux_sigtramp_frame_cache): Document bits/sigcontext.h
as where the offsets were obtained from.
* sparc64-linux-tdep.c (LINUX64_RT_SIGTRAMP_INSN0,
LINUX64_RT_SIGTRAMP_INSN1): New defines.
(sparc64_linux_sigtramp_start, sparc64_linux_sigtramp_p,
sparc64_linux_sigtramp_frame_cache,
sparc64_linux_sigtramp_frame_this_id,
sparc64_linux_sigtramp_frame_prev_register,
sparc64_linux_sigtramp_frame_sniffer): New functions.
(sparc64_linux_sigtramp_frame_unwind): New frame unwinder.
(sparc64_linux_init_abi): Register sigtramp frame sniffer.
* Makefile.in (sparc64-linux-tdep.o): Update dependencies.

* elfxx-sparc.h (struct _bfd_sparc_elf_link_hash_table): Remove
unsigned from type of dynamic_interpreter.
* elfxx-sparc.c (_bfd_sparc_elf_link_hash_table_create): Remove
casts again.

* elf.c (assign_section_numbers): Fix comment.

* som.c (som_bfd_print_private_bfd_data): Add explicit cast to long
for struct som_exec_auxhdr fields.

Update to ISO-C90 and fix formatting

* configure.ac: On MinGW, define USE_WIN32API and link with
-lws2_32.
* ser-tcp.c (<winsock2.h>): Include, for Windows.
(ETIMEDOUT): Define, for Windows.
(ioctl): Likewise.
(closesocket): Define, for POSIX.
(net_open): Adjust for differences in socket functions between
Windows and UNIX.
(net_close): Likweise.
(net_read_prim): New function.
(net_write_prim): Likewise.
(_initialize_ser_tcp): Initialize winsock.  Fill in read_prim and
write_prim.
* ser-unix.h (ser_unix_readcchar): Remove.
(ser_unix_read_prim): Declare.
(ser_unix_write_prim): Likewise.
* ser-unix.c (generic_readchar): Move to ser-base.c.
(ser_unix_wait_for): Likewise.
(do_unix_readchar): Likewise.
(ser_unix_readchar): Likewise.
(_initialize_ser_hardwire): Initialize read_prim and write_prim.
(ser_unix_read_prim): New function.
(ser_unix_write_prim): Likewise.
* ser-base.h (generic_readchar): Declare.
(ser_base_readchar): Likewise.
* ser-base.c (<winsock2.h>): Include, for windows.
(fd_event): Use the read primitive specified by the serial
interface.
(ser_base_wait_for): Moved from ser-unix.c
(do_ser_base_read_char): Likewise.
(generic_readchar): Likewise.
(ser_base_readchar): Likewise.
(ser_base_write): Use the write primitive specified by the serial
interface.
* ser-pipe.c (_initialize_ser_pipe): Use ser_base_readchar, not
ser_unix_readchar.  Initialize read_prim and write_prim.
* serial.c (struct serial_ops): Add read_prim and write_prim.
* configure: Regenerate.

daily update

*** empty log message ***

added missing header file #include "libiberty.h" to psim.c

2005-04-20  Manoj Iyer  <manjo@austin.ibm.com>

     * psim.c: Added libiberty.h header file.

added missing header file libiberty.h to psim.c

Insert a blank like after header

Inserting a blank line to changelog

* readelf.c (display_debug_frames): Use data factor for
DW_CFA_def_cfa_sf and DW_CFA_def_cfa_offset_sf.

* elfxx-sparc.c (sparc_elf_r_symndx_64): Fix compiler warning.
(_bfd_sparc_elf_link_hash_table_create): Likewise.
(allocate_dynrelocs): Likewise.

2005-04-20  H.J. Lu  <hongjiu.lu@intel.com>

* config/obj-aout.h (S_IS_EXTERN): Removed.
* config/obj-bout.h (S_IS_EXTERN): Likewise.
* config/obj-coff.h (S_IS_EXTERN): Likewise.
* symbols.c (S_IS_EXTERN): Likewise.
* symbols.h (S_IS_EXTERN): Likewise.

* config/tc-alpha.c (tc_gen_reloc): Replace S_IS_EXTERN with
S_IS_EXTERNAL.
* config/tc-d10v.c (md_apply_fix3): Likewise.
* config/tc-ia64.c (ia64_fix_adjustable): Likewise.
* config/tc-iq2000.c (iq2000_fix_adjustable): Likewise.
* config/tc-m32r.c (m32r_fix_adjustable): Likewise.
* config/tc-mmix.c (mmix_adjust_symtab): Likewise.
* config/tc-sh64.c (shmedia_frob_file_before_adjust): Likewise.
(shmedia_md_convert_frag): Likewise.
* symbols.c (print_symbol_value_1): Likewise.
* write.c (write_object_file): Likewise.

(get_aligned_diff): Change type of branch_align to offsetT so that its
signedness matches that of target_size.

unsigned32 and unsigned64 is multiply defined in sim-types.h.

2005-04-20  Manoj Iyer  <manjo@austin.ibm.com>
* sim-types.h: Changed unsigned32 and unsigned64 to signed32
and signed64 for __ALPHA__.

* configure.in: Fix the check for basename declaration. Add check
for declarations of ftello, ftello64, fseeko, fseeko64.
* configure: Regenerate.
* config.in: Ditto.
* sysdep.h: If needed, declare ftello, ftello64, fseeko, fseeko64.

Use HAVE_LONG_LONG to check if compiler supports long long instead of using __GNU__.

2005-04-18  Manoj Iyer  <manjo@austin.ibm.com>
* configure.ac: Added check for long long.
* config.in: Regenerated.
* configure: Regenerated.
* words.h: Modified logic to check for HAVE_LONG_LONG instead of
__GNUC__, added config.h header file.

(macro): Use sprintf_vma to convert a > 32 bit number into a readable string.
(load_register): Likewise.

* config/tc-ia64.c (specify_resource): Initialize all of tmpl.

gas/testsuite/
2005-04-20  Jan Beulich  <jbeulich@novell.com>

* gas/elf/struct.s: Adjust to not get into alignment issues.
* gas/elf/struct.d: Adjust for the above and the test's name.

daily update

*** empty log message ***

2005-04-19  H.J. Lu  <hongjiu.lu@intel.com>

* objdump.c (dump_section_header): Skip linker created section.

* elfxx-sparc.c: New file.
* elfxx-sparc.h: New file.
* Makefile.in (BFD32_BACKENDS): Add elfxx-sparc.lo
(BFD32_BACKENDS_CFILES): Add elfxx-sparc.c
(elfxx-sparc.lo): New rule.
(elf32-sparc.lo): Add elfxx-sparc.h dependency.
(elf64-sparc.lo): Likewise.
* configure.in (bfd_elf32_sparc_vec): Add elfxx-sparc.lo
(bfd_elf64_sparc_vec): Likewise.
* configure: Regenerate.
* elf32-sparc.c: Remove common code now in elfxx-sparc.c
* elf64-sparc.c: Likewise, also use elf64_sparc_*() naming
which is more consistent with elf32-sparc.c

* config/tc-sparc.c (md_assemble): If sparc_ip gives us a
NULL insn, exit early.  Remove now spurious NULL checks.
(sparc_ip): Use as_bad for unknown opcode errors, set *pinsn
to NULL and exit.

merge from gcc

* elfxx-mips.c (struct mips_elf_link_hash_entry): Update comment.
(mips_elf_create_local_got_entry): Check h->root.forced_local.

2005-04-19  H.J. Lu  <hongjiu.lu@intel.com>

* gas/i386/padlock.d: Updated.

gas/
2005-04-19  Jan Beulich  <jbeulich@novell.com>

* symbols.h (symbol_find_base): Remove prototype.
* symbols.c (save_symbol_name): Remove code section conditional upon
STRIP_UNDERSCORE.
(symbol_find): Remove.
(symbol_find_base): Rename to symbol_find.
* subsegs.c (section_symbol): Replace use of symbol_find_base with
symbol_find.
* config/obj-coff.c (tag_insert): Remove code section conditional
upon STRIP_UNDERSCORE.
(obj_coff_def): Likewise.
(obj_coff_endef): Replace use of symbol_find_base with symbol_find.
(coff_frob_symbol): Likewise.
(yank_symbols): Likewise.
(c_section_symbol): Likewise.
* config/obj-coff.h (DO_NOT_STRIP): Remove.
* config/tc-arm.c (symbol_locate): Remove code section conditional
upon STRIP_UNDERSCORE.
* config/tc-h8300.h (DO_NOT_STRIP): Remove.
* config/tc-h8500.h (DO_NOT_STRIP): Remove.
* config/tc-sh.h (DO_NOT_STRIP): Remove.
* config/tc-w65.h (DO_NOT_STRIP): Remove.
* config/tc-z8k.h (DO_NOT_STRIP): Remove.

* config/tc-ia64.c (md_assemble): Fix error message for wrong
access to application registers.

testsuite/:
* gas/ia64/invalid-ar.l: Adapt to changed error message.

gas/
2005-04-19  Jan Beulich  <jbeulich@novell.com>

* config/te-sysv32.h: Remove.
* Makefile.am (TARG_ENV_HFILES): Remove reference to
config/te-sysv32.h.
* Makefile.in (TARG_ENV_HFILES): Likewise.

gas/
2005-04-19  Jan Beulich  <jbeulich@novell.com>

PR/847
* config/tc-ia64.c (ia64_canonicalize_symbol_name): Re-allow zero-
length symbols.

* c-lang.c (c_create_fundamental_type): Comment fix.

bfd/
* elf32-ppc.c (ppc_elf_check_relocs): Don't bother with
dynamic relocs in non-SEC_ALLOC sections.
(ppc_elf_gc_sweep_hook): Likewise.
(ppc_elf_relax_section): Likewise.
(ppc_elf_relocate_section): Likewise.
ld/testsuite/
* ld-powerpc/reloc.d: Update to suit removal of non-alloc relocs.

* ppc-opc.c (RTO): Define.
(powerpc_opcodes <tlbsx, tlbsx., tlbre>): Combine PPC403 and BOOKE
entries to suit PPC440.

daily update

*** empty log message ***

dos2unix

2005-04-18  Michael Snyder  <msnyder@redhat.com>

* mn10300-tdep.c: Assimilate code from mn10300-prologue.c.
* mn10300-prologue.c: Remove.
* mn10300-tdep.h: Add enums from mn10300-prologue.c.
Remove prototypes.
* config/mn10300/mn10300.mt: Remove mn10300-prologue module.

gas/ChangeLog:
* config/tc-i386.c (md_begin): Allow hyphens in mnemonics.
include/opcode/ChangeLog:
* i386.h: Insert hyphens into selected VIA PadLock extensions.
Add xcrypt-ctr.  Provide aliases without hyphens.
opcodes/ChangeLog:
* i386-dis.c: Insert hyphens into selected VIA PadLock extensions.
Add xcrypt-ctr.

2005-04-18  Michael Snyder  <msnyder@redhat.com>

* config/mn10300/mn10300.mt: Add mn10300-prologue module.

2005-04-18  Michael Snyder  <msnyder@redhat.com>

* ia64-tdep.c (ia64_get_dyn_info_list): Fix indent.

* config/tc-mips.c (normalize_constant_expr): Don't check for
HAVE_32BIT_GPRS.
(check_absolute_expr): Only call normalize_constant_expr() if
HAVE_32BIT_GPRS.
(mips_ip): Likewise.

* config/tc-mips.c (check_absolute_expr): Fix formatting.

gas/
2005-04-18  Jan Beulich  <jbeulich@novell.com>

* macro.c (free_token): New, freeing all the memory associated with a
macro.
(do_formals): Move initializers to ...
(define_macro): ... here.
(delete_macro): Convert passed in name to lower case. Warn when
purging macro that doesn't exist. Use hash_jam instead of hash_delete.

gas/testsuite/
2005-04-18  Jan Beulich  <jbeulich@novell.com>

* gas/macros/purge.[ls]: New.
* gas/macros/macros.exp: Run new test.

(Backtrace): Describe 'bt full'.

*** empty log message ***

(WMAC, WMADD): Move casts from the LHS of an assignment operator to the RHS.
(WSLL, WSRA, WSRL, WUNPCKEH, WUNPACKEL): Use ULL suffix to indicate an
unsigned long long constant.

(xcoff64_core_p): Fix compile time warning assigning a value to return_value.