Apparently delete doesn't need to worry about nullptr so removing thi…

…s check.
substrait-io · Feb 9, 2024 · c28f6f0 · c28f6f0
1 parent 755c4ef
commit c28f6f0
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/export/planloader/planloader.cpp b/export/planloader/planloader.cpp
@@ -35,12 +35,8 @@ SerializedPlan* load_substrait_plan(const char* filename) {
 }
 
 void free_substrait_plan(SerializedPlan* plan) {
-  if (plan->buffer) {
-    delete[] plan->buffer;
-  }
-  if (plan->errorMessage) {
-    delete[] plan->errorMessage;
-  }
+  delete[] plan->buffer;
+  delete[] plan->errorMessage;
   delete plan;
 }