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, 4 insertions, 10 deletions

diff --git a/libqpdf/Pl_RC4.cc b/libqpdf/Pl_RC4.cc
index 87c17f83..407490ca 100644
--- a/libqpdf/Pl_RC4.cc
+++ b/libqpdf/Pl_RC4.cc
@@ -13,11 +13,8 @@ Pl_RC4::Pl_RC4(char const* identifier, Pipeline* next,
 
 Pl_RC4::~Pl_RC4()
 {
-    if (this->outbuf)
-    {
-	delete [] this->outbuf;
-	this->outbuf = 0;
-    }
+    delete [] this->outbuf;
+    this->outbuf = 0;
 }
 
 void
@@ -47,10 +44,7 @@ Pl_RC4::write(unsigned char* data, size_t len)
 void
 Pl_RC4::finish()
 {
-    if (this->outbuf)
-    {
-	delete [] this->outbuf;
-	this->outbuf = 0;
-    }
+    delete [] this->outbuf;
+    this->outbuf = 0;
     this->getNext()->finish();
 }