RE: [PATCH] gdb/testsuite: fix gdb.python/py-events.exp for finding process id

["Aktemur, Tankut Baris" <tankut.baris.aktemur@intel.com>]

On Wednesday, January 12, 2022 3:50 PM, Andrew Burgess wrote:
> > diff --git a/gdb/testsuite/gdb.python/py-events.exp b/gdb/testsuite/gdb.python/py-
> events.exp
> > index 4df012b943b..4b411999588 100644
> > --- a/gdb/testsuite/gdb.python/py-events.exp
> > +++ b/gdb/testsuite/gdb.python/py-events.exp
> > @@ -81,12 +81,17 @@ delete_breakpoints
> >
> >  # Test inferior call events
> >
> > -gdb_test_multiple "info threads" "get current thread" {
> > -    -re -wrap "process ($decimal)\[^\n\r\]*do_nothing.*" {
> > +set process_id 0
> 
> This looks good.  My only suggestion would be don't initialise
> process_id to a number.  I know it's pretty unlikely that `0` is ever
> going to be a valid result, but it's not impossible.
> 
> If you instead initialise to something like "invalid" then there's
> even less chance that this will match the actual output.
> 
> With that change this is good to apply.
> 
> Thanks,
> Andrew

Pushed with this fix.

Thanks.
-Baris


Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928