public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
* Testsuite regression
@ 2000-03-26 10:23 Mark Kettenis
  2000-03-26 10:42 ` Daniel Berlin+list.gdb
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Mark Kettenis @ 2000-03-26 10:23 UTC (permalink / raw)
  To: gdb

Hi all,

Somewhere between March 9 and March 15, the following failure appears:

  FAIL: gdb.base/printcmds.exp: p &ctable2[15*16] with print elements set to 16

This is the output from a run where the test still passed:

  p &ctable2[15*16]
  $538 = (unsigned char *) 'a' <repeats 16 times>

And this the current output:

  p &ctable2[15*16]
  $538 = (unsigned char *) 'a' <repeats 16 times>...

I'm a bit puzzled though what change is responsible for this
regression.  Is there anybody else observing this failure?

Mark

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

* Re: Testsuite regression
  2000-03-26 10:23 Testsuite regression Mark Kettenis
@ 2000-03-26 10:42 ` Daniel Berlin+list.gdb
  2000-04-01  0:00   ` Daniel Berlin+list.gdb
  2000-03-26 11:41 ` Peter.Schauer
  2000-04-01  0:00 ` Mark Kettenis
  2 siblings, 1 reply; 10+ messages in thread
From: Daniel Berlin+list.gdb @ 2000-03-26 10:42 UTC (permalink / raw)
  To: gdb

Mark Kettenis <kettenis@wins.uva.nl> writes:

> I'm a bit puzzled though what change is responsible for this
> regression.  Is there anybody else observing this failure?
I see it too, i have no idea what is causing it.
I think it's expecting another space before the ellipsis or a comma,
or something.

> 
> Mark

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

* Re: Testsuite regression
  2000-03-26 10:23 Testsuite regression Mark Kettenis
  2000-03-26 10:42 ` Daniel Berlin+list.gdb
@ 2000-03-26 11:41 ` Peter.Schauer
  2000-03-27  9:36   ` Elena Zannoni
  2000-04-01  0:00   ` Peter.Schauer
  2000-04-01  0:00 ` Mark Kettenis
  2 siblings, 2 replies; 10+ messages in thread
From: Peter.Schauer @ 2000-03-26 11:41 UTC (permalink / raw)
  To: Mark Kettenis; +Cc: gdb

I noticed this to, it is caused by:

2000-03-14  Elena Zannoni  <ezannoni@kwikemart.cygnus.com>

        * gdb.base/printcmds.c: Add typedeffed arrays.

which now puts a nonzero word after ctable2 in gdb.base/printcmds.c via
ArrayInt a1 = {2,4,6,8,10,12,14,16,18,20};

Previous versions had
int int1dim[12] = {0,1,2,3,4,5,6,7,8,9,10,11};
after ctable2, putting a zero word there.

So we now have a non zero byte after ctable2 (but only on little endian
targets).

p &ctable2[15*16]
asks GDB to print an unsigned char pointer and GDB puts out the contents
of the pointer as a string as well. As the string is no longer zero
terminated, GDB appends ellipsis.

It could be fixed by appending a zero byte to ctable2 (but I have tested
this only lightly):

*** gdb/testsuite/gdb.base/printcmds.c.orig	Wed Mar 22 19:08:22 2000
--- gdb/testsuite/gdb.base/printcmds.c	Sun Mar 26 21:34:20 2000
***************
*** 53,59 ****
    'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
!   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a'
  };
  
  /* Single and multidimensional arrays to test access and printing of array
--- 53,59 ----
    'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
!   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a', 0
  };
  
  /* Single and multidimensional arrays to test access and printing of array

> Hi all,
> 
> Somewhere between March 9 and March 15, the following failure appears:
> 
>   FAIL: gdb.base/printcmds.exp: p &ctable2[15*16] with print elements set to 16
> 
> This is the output from a run where the test still passed:
> 
>   p &ctable2[15*16]
>   $538 = (unsigned char *) 'a' <repeats 16 times>
> 
> And this the current output:
> 
>   p &ctable2[15*16]
>   $538 = (unsigned char *) 'a' <repeats 16 times>...
> 
> I'm a bit puzzled though what change is responsible for this
> regression.  Is there anybody else observing this failure?
> 
> Mark

-- 
Peter Schauer			pes@regent.e-technik.tu-muenchen.de

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

* Re: Testsuite regression
  2000-03-26 11:41 ` Peter.Schauer
@ 2000-03-27  9:36   ` Elena Zannoni
  2000-03-27 16:00     ` Stan Shebs
  2000-04-01  0:00     ` Elena Zannoni
  2000-04-01  0:00   ` Peter.Schauer
  1 sibling, 2 replies; 10+ messages in thread
From: Elena Zannoni @ 2000-03-27  9:36 UTC (permalink / raw)
  To: Peter.Schauer; +Cc: Mark Kettenis, gdb

I tested your fix on solaris and linux, it seems to work fine. I have
committed it. (Sorry, I know I shouldn't have done it w/o official
approval from Stan).

Elena

Peter.Schauer writes:
 > I noticed this to, it is caused by:
 > 
 > 2000-03-14  Elena Zannoni  <ezannoni@kwikemart.cygnus.com>
 > 
 >         * gdb.base/printcmds.c: Add typedeffed arrays.
 > 
 > which now puts a nonzero word after ctable2 in gdb.base/printcmds.c via
 > ArrayInt a1 = {2,4,6,8,10,12,14,16,18,20};
 > 
 > Previous versions had
 > int int1dim[12] = {0,1,2,3,4,5,6,7,8,9,10,11};
 > after ctable2, putting a zero word there.
 > 
 > So we now have a non zero byte after ctable2 (but only on little endian
 > targets).
 > 
 > p &ctable2[15*16]
 > asks GDB to print an unsigned char pointer and GDB puts out the contents
 > of the pointer as a string as well. As the string is no longer zero
 > terminated, GDB appends ellipsis.
 > 
 > It could be fixed by appending a zero byte to ctable2 (but I have tested
 > this only lightly):
 > 
 > *** gdb/testsuite/gdb.base/printcmds.c.orig	Wed Mar 22 19:08:22 2000
 > --- gdb/testsuite/gdb.base/printcmds.c	Sun Mar 26 21:34:20 2000
 > ***************
 > *** 53,59 ****
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
 > !   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a'
 >   };
 >   
 >   /* Single and multidimensional arrays to test access and printing of array
 > --- 53,59 ----
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
 > !   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a', 0
 >   };
 >   
 >   /* Single and multidimensional arrays to test access and printing of array
 > 
 > > Hi all,
 > > 
 > > Somewhere between March 9 and March 15, the following failure appears:
 > > 
 > >   FAIL: gdb.base/printcmds.exp: p &ctable2[15*16] with print elements set to 16
 > > 
 > > This is the output from a run where the test still passed:
 > > 
 > >   p &ctable2[15*16]
 > >   $538 = (unsigned char *) 'a' <repeats 16 times>
 > > 
 > > And this the current output:
 > > 
 > >   p &ctable2[15*16]
 > >   $538 = (unsigned char *) 'a' <repeats 16 times>...
 > > 
 > > I'm a bit puzzled though what change is responsible for this
 > > regression.  Is there anybody else observing this failure?
 > > 
 > > Mark
 > 
 > -- 
 > Peter Schauer			pes@regent.e-technik.tu-muenchen.de

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

* Re: Testsuite regression
  2000-03-27  9:36   ` Elena Zannoni
@ 2000-03-27 16:00     ` Stan Shebs
  2000-04-01  0:00       ` Stan Shebs
  2000-04-01  0:00     ` Elena Zannoni
  1 sibling, 1 reply; 10+ messages in thread
From: Stan Shebs @ 2000-03-27 16:00 UTC (permalink / raw)
  To: Elena Zannoni; +Cc: Peter.Schauer, Mark Kettenis, gdb

Elena Zannoni wrote:
> 
> I tested your fix on solaris and linux, it seems to work fine. I have
> committed it. (Sorry, I know I shouldn't have done it w/o official
> approval from Stan).

No, that's cool.  To quote from the GCC pages, "We don't want to get overly
anal about checkin policies".

Stan

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

* Re: Testsuite regression
  2000-03-26 11:41 ` Peter.Schauer
  2000-03-27  9:36   ` Elena Zannoni
@ 2000-04-01  0:00   ` Peter.Schauer
  1 sibling, 0 replies; 10+ messages in thread
From: Peter.Schauer @ 2000-04-01  0:00 UTC (permalink / raw)
  To: Mark Kettenis; +Cc: gdb

I noticed this to, it is caused by:

2000-03-14  Elena Zannoni  <ezannoni@kwikemart.cygnus.com>

        * gdb.base/printcmds.c: Add typedeffed arrays.

which now puts a nonzero word after ctable2 in gdb.base/printcmds.c via
ArrayInt a1 = {2,4,6,8,10,12,14,16,18,20};

Previous versions had
int int1dim[12] = {0,1,2,3,4,5,6,7,8,9,10,11};
after ctable2, putting a zero word there.

So we now have a non zero byte after ctable2 (but only on little endian
targets).

p &ctable2[15*16]
asks GDB to print an unsigned char pointer and GDB puts out the contents
of the pointer as a string as well. As the string is no longer zero
terminated, GDB appends ellipsis.

It could be fixed by appending a zero byte to ctable2 (but I have tested
this only lightly):

*** gdb/testsuite/gdb.base/printcmds.c.orig	Wed Mar 22 19:08:22 2000
--- gdb/testsuite/gdb.base/printcmds.c	Sun Mar 26 21:34:20 2000
***************
*** 53,59 ****
    'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
!   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a'
  };
  
  /* Single and multidimensional arrays to test access and printing of array
--- 53,59 ----
    'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
    'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
!   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a', 0
  };
  
  /* Single and multidimensional arrays to test access and printing of array

> Hi all,
> 
> Somewhere between March 9 and March 15, the following failure appears:
> 
>   FAIL: gdb.base/printcmds.exp: p &ctable2[15*16] with print elements set to 16
> 
> This is the output from a run where the test still passed:
> 
>   p &ctable2[15*16]
>   $538 = (unsigned char *) 'a' <repeats 16 times>
> 
> And this the current output:
> 
>   p &ctable2[15*16]
>   $538 = (unsigned char *) 'a' <repeats 16 times>...
> 
> I'm a bit puzzled though what change is responsible for this
> regression.  Is there anybody else observing this failure?
> 
> Mark

-- 
Peter Schauer			pes@regent.e-technik.tu-muenchen.de

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

* Re: Testsuite regression
  2000-03-27 16:00     ` Stan Shebs
@ 2000-04-01  0:00       ` Stan Shebs
  0 siblings, 0 replies; 10+ messages in thread
From: Stan Shebs @ 2000-04-01  0:00 UTC (permalink / raw)
  To: Elena Zannoni; +Cc: Peter.Schauer, Mark Kettenis, gdb

Elena Zannoni wrote:
> 
> I tested your fix on solaris and linux, it seems to work fine. I have
> committed it. (Sorry, I know I shouldn't have done it w/o official
> approval from Stan).

No, that's cool.  To quote from the GCC pages, "We don't want to get overly
anal about checkin policies".

Stan

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

* Testsuite regression
  2000-03-26 10:23 Testsuite regression Mark Kettenis
  2000-03-26 10:42 ` Daniel Berlin+list.gdb
  2000-03-26 11:41 ` Peter.Schauer
@ 2000-04-01  0:00 ` Mark Kettenis
  2 siblings, 0 replies; 10+ messages in thread
From: Mark Kettenis @ 2000-04-01  0:00 UTC (permalink / raw)
  To: gdb

Hi all,

Somewhere between March 9 and March 15, the following failure appears:

  FAIL: gdb.base/printcmds.exp: p &ctable2[15*16] with print elements set to 16

This is the output from a run where the test still passed:

  p &ctable2[15*16]
  $538 = (unsigned char *) 'a' <repeats 16 times>

And this the current output:

  p &ctable2[15*16]
  $538 = (unsigned char *) 'a' <repeats 16 times>...

I'm a bit puzzled though what change is responsible for this
regression.  Is there anybody else observing this failure?

Mark

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

* Re: Testsuite regression
  2000-03-26 10:42 ` Daniel Berlin+list.gdb
@ 2000-04-01  0:00   ` Daniel Berlin+list.gdb
  0 siblings, 0 replies; 10+ messages in thread
From: Daniel Berlin+list.gdb @ 2000-04-01  0:00 UTC (permalink / raw)
  To: gdb

Mark Kettenis <kettenis@wins.uva.nl> writes:

> I'm a bit puzzled though what change is responsible for this
> regression.  Is there anybody else observing this failure?
I see it too, i have no idea what is causing it.
I think it's expecting another space before the ellipsis or a comma,
or something.

> 
> Mark

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

* Re: Testsuite regression
  2000-03-27  9:36   ` Elena Zannoni
  2000-03-27 16:00     ` Stan Shebs
@ 2000-04-01  0:00     ` Elena Zannoni
  1 sibling, 0 replies; 10+ messages in thread
From: Elena Zannoni @ 2000-04-01  0:00 UTC (permalink / raw)
  To: Peter.Schauer; +Cc: Mark Kettenis, gdb

I tested your fix on solaris and linux, it seems to work fine. I have
committed it. (Sorry, I know I shouldn't have done it w/o official
approval from Stan).

Elena

Peter.Schauer writes:
 > I noticed this to, it is caused by:
 > 
 > 2000-03-14  Elena Zannoni  <ezannoni@kwikemart.cygnus.com>
 > 
 >         * gdb.base/printcmds.c: Add typedeffed arrays.
 > 
 > which now puts a nonzero word after ctable2 in gdb.base/printcmds.c via
 > ArrayInt a1 = {2,4,6,8,10,12,14,16,18,20};
 > 
 > Previous versions had
 > int int1dim[12] = {0,1,2,3,4,5,6,7,8,9,10,11};
 > after ctable2, putting a zero word there.
 > 
 > So we now have a non zero byte after ctable2 (but only on little endian
 > targets).
 > 
 > p &ctable2[15*16]
 > asks GDB to print an unsigned char pointer and GDB puts out the contents
 > of the pointer as a string as well. As the string is no longer zero
 > terminated, GDB appends ellipsis.
 > 
 > It could be fixed by appending a zero byte to ctable2 (but I have tested
 > this only lightly):
 > 
 > *** gdb/testsuite/gdb.base/printcmds.c.orig	Wed Mar 22 19:08:22 2000
 > --- gdb/testsuite/gdb.base/printcmds.c	Sun Mar 26 21:34:20 2000
 > ***************
 > *** 53,59 ****
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
 > !   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a'
 >   };
 >   
 >   /* Single and multidimensional arrays to test access and printing of array
 > --- 53,59 ----
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','X','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','X','X',
 >     'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','X',
 > !   'a','a','a','a','a','a','a','a','a','a','a','a','a','a','a','a', 0
 >   };
 >   
 >   /* Single and multidimensional arrays to test access and printing of array
 > 
 > > Hi all,
 > > 
 > > Somewhere between March 9 and March 15, the following failure appears:
 > > 
 > >   FAIL: gdb.base/printcmds.exp: p &ctable2[15*16] with print elements set to 16
 > > 
 > > This is the output from a run where the test still passed:
 > > 
 > >   p &ctable2[15*16]
 > >   $538 = (unsigned char *) 'a' <repeats 16 times>
 > > 
 > > And this the current output:
 > > 
 > >   p &ctable2[15*16]
 > >   $538 = (unsigned char *) 'a' <repeats 16 times>...
 > > 
 > > I'm a bit puzzled though what change is responsible for this
 > > regression.  Is there anybody else observing this failure?
 > > 
 > > Mark
 > 
 > -- 
 > Peter Schauer			pes@regent.e-technik.tu-muenchen.de

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

end of thread, other threads:[~2000-04-01  0:00 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2000-03-26 10:23 Testsuite regression Mark Kettenis
2000-03-26 10:42 ` Daniel Berlin+list.gdb
2000-04-01  0:00   ` Daniel Berlin+list.gdb
2000-03-26 11:41 ` Peter.Schauer
2000-03-27  9:36   ` Elena Zannoni
2000-03-27 16:00     ` Stan Shebs
2000-04-01  0:00       ` Stan Shebs
2000-04-01  0:00     ` Elena Zannoni
2000-04-01  0:00   ` Peter.Schauer
2000-04-01  0:00 ` Mark Kettenis

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