diff options
author | Tom Tromey <tromey@redhat.com> | 2013-05-30 17:39:34 +0000 |
---|---|---|
committer | Tom Tromey <tromey@redhat.com> | 2013-05-30 17:39:34 +0000 |
commit | db26349c64fd5303eb52612305271e4eddae087e (patch) | |
tree | 25997c107fe75e5e2200a6b88e7e4dbbc33e8288 /gdb/ChangeLog | |
parent | 77f9e713027d32cc5ca1fb8d05e7b1b3f5397ec7 (diff) | |
download | gdb-db26349c64fd5303eb52612305271e4eddae087e.zip gdb-db26349c64fd5303eb52612305271e4eddae087e.tar.gz gdb-db26349c64fd5303eb52612305271e4eddae087e.tar.bz2 |
fix compile_rx_or_error
compile_rx_or_error looks like a constructor, but it can return NULL.
This patch changes it to remove the NULL return, making it work
like any other cleanup constructor.
This is a stylistic patch but I think it is also better for code to
follow the normal conventions.
* probe.c (collect_probes): Check arguments for NULL before
calling compile_rx_or_error.
* utils.c (compile_rx_or_error): Require 'rx' to be non-NULL.
Remove NULL return.
Diffstat (limited to 'gdb/ChangeLog')
-rw-r--r-- | gdb/ChangeLog | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 7adba77..142efca 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,12 @@ 2013-05-30 Tom Tromey <tromey@redhat.com> + * probe.c (collect_probes): Check arguments for NULL before + calling compile_rx_or_error. + * utils.c (compile_rx_or_error): Require 'rx' to be non-NULL. + Remove NULL return. + +2013-05-30 Tom Tromey <tromey@redhat.com> + * infrun.c (adjust_pc_after_break): Introduce an outer null cleanup. |