From: Andrew Burgess Date: Wed, 5 Oct 2022 09:21:02 +0000 (+0100) Subject: gdb: make use of scoped_restore in unduplicated_should_be_inserted X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=aaa141a0a99fb63f6209da75257c58d94404a963;p=binutils-gdb.git gdb: make use of scoped_restore in unduplicated_should_be_inserted I noticed that we could make use of a scoped_restore in the function unduplicated_should_be_inserted. I've also converted the function return type from int to bool. This change shouldn't make any difference, as I don't think anything within should_be_inserted could throw an exception, but the change doesn't hurt, and will help keep us safe if anything ever changes in the future. Approved-By: Simon Marchi --- diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 84eeccd0cd7..bfa017f1398 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -2299,16 +2299,13 @@ should_be_inserted (struct bp_location *bl) /* Same as should_be_inserted but does the check assuming that the location is not duplicated. */ -static int +static bool unduplicated_should_be_inserted (struct bp_location *bl) { - int result; - const int save_duplicate = bl->duplicate; + scoped_restore restore_bl_duplicate + = make_scoped_restore (&bl->duplicate, 0); - bl->duplicate = 0; - result = should_be_inserted (bl); - bl->duplicate = save_duplicate; - return result; + return should_be_inserted (bl); } /* Parses a conditional described by an expression COND into an