[RFA/Semi-obvious/5.3?] Fix compilation error in hppa-tdep.c

[Joel Brobecker <brobecker@gnat.com>]

[-- Attachment #1: Type: text/plain, Size: 1121 bytes --]

I guess there was a slight improvement in the profile of
find_function_in_inferior. This causes a compilation failure on HP/UX.

As I was working on this file, I noticed a warning saying that we should
an extra pair of braces around the "&" expression, to avoid any
confusion. So I added it. I sometimes make silly mistakes, so I would
appreciate if somebody had a quick look, to make sure I did not screw
up.

This also gives me the opportunity to announce that the HP/UX target
is important for ACT, and that we will do the work necessary to keep
this port in good shape. My first goal for this platform is to make it
at least multiarch partial.

2002-10-30  Joel Brobecker  <brobecker@gnat.com>

        * hppa-tdep.c (find_function_in_inferior): Fix the declaration
        that caused a compilation error on HP/UX.
        (low_text_segment_addres): Fix a compilation warning.

Ok to apply?

Also, I am not sure about the 5.3 branch, I haven't pulled the sources
from this branch yet. But I will check this shortly and apply the fix
there if necessary (provided the change is approved, naturally).

Thanks,
-- 
Joel

[-- Attachment #2: hppa-tdep.c.diff --]
[-- Type: text/plain, Size: 1536 bytes --]

Index: hppa-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/hppa-tdep.c,v
retrieving revision 1.28
diff -c -3 -p -r1.28 hppa-tdep.c
*** hppa-tdep.c	24 Oct 2002 21:02:53 -0000	1.28
--- hppa-tdep.c	30 Oct 2002 07:57:35 -0000
*************** extern int hp_som_som_object_present;
*** 150,156 ****
  extern int exception_catchpoints_are_fragile;
  
  /* This is defined in valops.c. */
! extern struct value *find_function_in_inferior (char *);
  
  /* Should call_function allocate stack space for a struct return?  */
  int
--- 150,156 ----
  extern int exception_catchpoints_are_fragile;
  
  /* This is defined in valops.c. */
! extern struct value *find_function_in_inferior (const char *);
  
  /* Should call_function allocate stack space for a struct return?  */
  int
*************** static CORE_ADDR low_text_segment_addres
*** 300,306 ****
  static void
  record_text_segment_lowaddr (bfd *abfd, asection *section, void *ignored)
  {
!   if ((section->flags & (SEC_ALLOC | SEC_LOAD | SEC_READONLY)
         == (SEC_ALLOC | SEC_LOAD | SEC_READONLY))
        && section->vma < low_text_segment_address)
      low_text_segment_address = section->vma;
--- 300,306 ----
  static void
  record_text_segment_lowaddr (bfd *abfd, asection *section, void *ignored)
  {
!   if (((section->flags & (SEC_ALLOC | SEC_LOAD | SEC_READONLY))
         == (SEC_ALLOC | SEC_LOAD | SEC_READONLY))
        && section->vma < low_text_segment_address)
      low_text_segment_address = section->vma;