diff options
| author | Tom Tromey <tom@tromey.com> | 2022-05-28 21:06:19 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2022-07-28 14:16:50 -0600 |
| commit | 43cffa64cf72f11a95203cd89844f82aa906c7e7 (patch) | |
| tree | 0d8a792fa7a0ea3f169345e2b9ae739c83af58b4 /gdb/guile/scm-block.c | |
| parent | 8126c055e453a494a72a44c9cce39fa8ddf14607 (diff) | |
| download | binutils-43cffa64cf72f11a95203cd89844f82aa906c7e7.zip binutils-43cffa64cf72f11a95203cd89844f82aa906c7e7.tar.gz binutils-43cffa64cf72f11a95203cd89844f82aa906c7e7.tar.bz2 | |
Remove some unneeded checks in Guile code
The Guile code generally checks to see if an htab is non-null before
destroying it. However, the registry code already ensures this, so we
can change these checks to asserts and simplify the code a little.
Diffstat (limited to 'gdb/guile/scm-block.c')
| -rw-r--r-- | gdb/guile/scm-block.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/gdb/guile/scm-block.c b/gdb/guile/scm-block.c index a29c2db..07c6a2b 100644 --- a/gdb/guile/scm-block.c +++ b/gdb/guile/scm-block.c @@ -97,11 +97,9 @@ struct bkscm_deleter void operator() (htab_t htab) { - if (htab != NULL) - { - htab_traverse_noresize (htab, bkscm_mark_block_invalid, NULL); - htab_delete (htab); - } + gdb_assert (htab != nullptr); + htab_traverse_noresize (htab, bkscm_mark_block_invalid, NULL); + htab_delete (htab); } }; |