[nasm:machofix] codeview.c: remove superfluous pointer test

[nasm-bot f. H. P. A. <hp...@li...>]

Commit-ID:  5482a15e17cfc80379052253003e5a6b0cd46f14
Gitweb:     http://repo.or.cz/w/nasm.git?a=commitdiff;h=5482a15e17cfc80379052253003e5a6b0cd46f14
Author:     H. Peter Anvin <hp...@li...>
AuthorDate: Fri, 5 Feb 2016 11:31:46 -0800
Committer:  H. Peter Anvin <hp...@li...>
CommitDate: Fri, 5 Feb 2016 11:31:46 -0800

codeview.c: remove superfluous pointer test

A pointer to a field in a structure can never be NULL so remove
the test.  There is no reason to test the field for an empty string
either, since we immediately thereafter do a strcmp().

Signed-off-by: H. Peter Anvin <hp...@li...>


---
 output/codeview.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/output/codeview.c b/output/codeview.c
index 4256e8d..2f5bb69 100644
--- a/output/codeview.c
+++ b/output/codeview.c
@@ -1,6 +1,6 @@
 /* ----------------------------------------------------------------------- *
  *
- *   Copyright 1996-2010 The NASM Authors - All Rights Reserved
+ *   Copyright 1996-2016 The NASM Authors - All Rights Reserved
  *   See the file AUTHORS included with the NASM distribution for
  *   the specific copyright holders.
  *
@@ -402,7 +402,7 @@ static void register_reloc(struct coff_Section *const sect,
     for (uint32_t i = 0; i < coff_nsyms; i++) {
         struct coff_Symbol *s = saa_rstruct(coff_syms);
         r->symbol++;
-        if (s->strpos == -1 && s->name && !strcmp(sym, s->name)) {
+        if (s->strpos == -1 && !strcmp(sym, s->name)) {
             return;
         } else if (s->strpos != -1) {
             int res;