Mercurial > hg
comparison mercurial/cext/bdiff.c @ 38301:d9e87566f879 stable
cext: stop worrying and love the free(NULL)
There is no need to check for a NULL pointer before calling free since
free(NULL) is defined by C standards as a no-op. Lots of software relies on
this behavior so it is completely safe to call even on the most obscure of
systems.
author | Josef 'Jeff' Sipek <jeffpc@josefsipek.net> |
---|---|
date | Wed, 13 Jun 2018 10:37:39 -0400 |
parents | 273ea09f6550 |
children | 93b812d5b818 |
comparison
equal
deleted
inserted
replaced
38292:6cc5d01a58a6 | 38301:d9e87566f879 |
---|---|
153 cleanup: | 153 cleanup: |
154 if (_save) | 154 if (_save) |
155 PyEval_RestoreThread(_save); | 155 PyEval_RestoreThread(_save); |
156 PyBuffer_Release(&ba); | 156 PyBuffer_Release(&ba); |
157 PyBuffer_Release(&bb); | 157 PyBuffer_Release(&bb); |
158 if (al) { | 158 free(al); |
159 free(al); | 159 free(bl); |
160 } | |
161 if (bl) { | |
162 free(bl); | |
163 } | |
164 if (l.next) { | 160 if (l.next) { |
165 bdiff_freehunks(l.next); | 161 bdiff_freehunks(l.next); |
166 } | 162 } |
167 return result; | 163 return result; |
168 } | 164 } |