diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2015-03-16 15:01:15 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2015-03-16 15:01:15 +0100 |
| commit | 0f4a03cbb6fdb84d05cb6aafe50444edad4f4119 (patch) | |
| tree | 89472eece4173a97ac3b80aba5e35ed70cdd7845 /lib/memdebug.h | |
| parent | 9e661601feba03d1158ac466a457d5a6ce7f3f11 (diff) | |
| download | curl-0f4a03cbb6fdb84d05cb6aafe50444edad4f4119.tar.gz | |
free: instead of Curl_safefree()
Since we just started make use of free(NULL) in order to simplify code,
this change takes it a step further and:
- converts lots of Curl_safefree() calls to good old free()
- makes Curl_safefree() not check the pointer before free()
The (new) rule of thumb is: if you really want a function call that
frees a pointer and then assigns it to NULL, then use Curl_safefree().
But we will prefer just using free() from now on.
Diffstat (limited to 'lib/memdebug.h')
| -rw-r--r-- | lib/memdebug.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/memdebug.h b/lib/memdebug.h index bd565c8dc..b2b0e6966 100644 --- a/lib/memdebug.h +++ b/lib/memdebug.h @@ -8,7 +8,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2013, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 1998 - 2015, Daniel Stenberg, <daniel@haxx.se>, et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -171,6 +171,6 @@ CURL_EXTERN int curl_fclose(FILE *file, int line, const char *source); */ #define Curl_safefree(ptr) \ - do {if((ptr)) {free((ptr)); (ptr) = NULL;}} WHILE_FALSE + do { free((ptr)); (ptr) = NULL;} WHILE_FALSE #endif /* HEADER_CURL_MEMDEBUG_H */ |