diff options
| author |  Tomaz Canabrava <tomaz.canabrava@intel.com> | 2014-05-12 14:58:15 -0300 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2014-05-13 09:26:50 +0900 |
| commit | c86d055db77cdeef6049449ac79fe404c40d28cc (patch) | |
| tree | f0e53f8d42567a670567e7600de287dcdde5b627 /qt-gui.cpp | |
| parent | e0c0ac5d5c7c218011278fbe57da291a7e10c8eb (diff) | |
| download | subsurface-c86d055db77cdeef6049449ac79fe404c40d28cc.tar.gz | |
Do not check for null before free.
C specs says that we can safelly free a NULL pointer, so there's no reason
to check if it's null before freeing it.
Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-gui.cpp')
| -rw-r--r-- | qt-gui.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/qt-gui.cpp b/qt-gui.cpp index 868389eab..9d6dde45f 100644 --- a/qt-gui.cpp +++ b/qt-gui.cpp @@ -165,10 +165,8 @@ void exit_ui(void) { delete window; delete application; - if (existing_filename) - free((void *)existing_filename); - if (default_dive_computer_device) - free((void *)default_dive_computer_device); + free((void *)existing_filename); + free((void *)default_dive_computer_device); } void set_filename(const char *filename, bool force) |