philippelh/bitchx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove unnecessary NULL checks in expr.c

Browse Source 
The first is unnecessary because after_expand() never returns NULL, and the 'rest' pointer has been
dereferenced multiple times anyway.

The second is unnecessary because parse_number(&ptr) cannot set ptr to NULL.
This commit is contained in:
Kevin Easton
2016-09-30 23:44:15 +10:00
parent 7e3c39a464
commit e2b351b9ce
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
source/expr.c
View File

@@ -1703,8 +1703,7 @@ register unsigned char c;
TruncateAndQuote(buffer, val, length, quote_em, pad_char); TruncateAndQuote(buffer, val, length, quote_em, pad_char);
new_free(&val); new_free(&val);
new_free(&sub_buffer); new_free(&sub_buffer);
if (rest) *rest = c2;
*rest = c2;
return rest; return rest;
} }
@@ -1789,7 +1788,7 @@ register unsigned char c;
TruncateAndQuote(buffer, tmp2, length, quote_em, pad_char); TruncateAndQuote(buffer, tmp2, length, quote_em, pad_char);
new_free(&tmp2); new_free(&tmp2);
return (ptr ? ptr : empty_string); return ptr;
} }
/* /*