From: Tom Tromey <tom@tromey.com>
Date: Fri, 30 Sep 2016 21:43:40 +0000 (-0600)
Subject: PR gdb/20653 - small cleanup in string_to_explicit_location
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e742d386c480efa5229bc8b1887a7718f840a457;p=binutils-gdb.git

PR gdb/20653 - small cleanup in string_to_explicit_location

This bug points out that string_to_explicit_location compares a char*
against '\0'; whereas comparing against NULL is more normal.

2016-10-05  Tom Tromey  <tom@tromey.com>

	PR breakpoints/20653:
	* location.c (string_to_explicit_location): Use NULL, not '\0'.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1fc318ef7b4..225bf755900 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2016-10-05  Tom Tromey  <tom@tromey.com>
+
+	PR breakpoints/20653:
+	* location.c (string_to_explicit_location): Use NULL, not '\0'.
+
 2016-10-05  Tom Tromey  <tom@tromey.com>
 
 	PR symtab/20652:
diff --git a/gdb/location.c b/gdb/location.c
index 65116c732f7..8dce21ada12 100644
--- a/gdb/location.c
+++ b/gdb/location.c
@@ -524,7 +524,7 @@ string_to_explicit_location (const char **argp,
      character is an explicit location.  "-p" is reserved, though,
      for probe locations.  */
   if (argp == NULL
-      || *argp == '\0'
+      || *argp == NULL
       || *argp[0] != '-'
       || !isalpha ((*argp)[1])
       || ((*argp)[0] == '-' && (*argp)[1] == 'p'))