Remove some extraneous null pointer checks (fixes #234)

There were a few places in the code that were checking that a pointer
wasn't null before deleting it, even though C++ has always allowed
delete 0. Most of the code did not perform these checks.

1 files changed, 1 insertions, 4 deletions

diff --git a/libqpdf/ClosedFileInputSource.cc b/libqpdf/ClosedFileInputSource.cc
index 63357886..e9c9b3bd 100644
--- a/libqpdf/ClosedFileInputSource.cc
+++ b/libqpdf/ClosedFileInputSource.cc
@@ -11,10 +11,7 @@ ClosedFileInputSource::Members::Members(char const* filename) :
 
 ClosedFileInputSource::Members::~Members()
 {
-    if (fis)
-    {
-        delete fis;
-    }
+    delete fis;
 }
 
 ClosedFileInputSource::ClosedFileInputSource(char const* filename) :