Check buffer emptiness

Author: Hind-M

include/sparrow_ipc/deserialize_primitive_array.hpp

@@ -43,16 +43,7 @@ namespace sparrow_ipc
 
         if (compression)
         {
-            // TODO Handle buffers emptiness thoroughly / which is and which is not allowed...
-            // Validity buffers can be empty
-            if (validity_buffer_span.empty())
-            {
-                buffers.push_back(validity_buffer_span);
-            }
-            else
-            {
-                buffers.push_back(utils::get_decompressed_buffer(validity_buffer_span, compression));
-            }
+            buffers.push_back(utils::get_decompressed_buffer(validity_buffer_span, compression));
             buffers.push_back(utils::get_decompressed_buffer(data_buffer_span, compression));
         }
         else

include/sparrow_ipc/deserialize_variable_size_binary_array.hpp

@@ -41,15 +41,7 @@ namespace sparrow_ipc
 
         if (compression)
         {
-            // Validity buffers can be empty
-            if (validity_buffer_span.empty())
-            {
-                buffers.push_back(validity_buffer_span);
-            }
-            else
-            {
-                buffers.push_back(utils::get_decompressed_buffer(validity_buffer_span, compression));
-            }
+            buffers.push_back(utils::get_decompressed_buffer(validity_buffer_span, compression));
             buffers.push_back(utils::get_decompressed_buffer(offset_buffer_span, compression));
             buffers.push_back(utils::get_decompressed_buffer(data_buffer_span, compression));
         }

src/deserialize_utils.cpp

@@ -64,7 +64,7 @@ namespace sparrow_ipc::utils
         const org::apache::arrow::flatbuf::BodyCompression* compression
     )
     {
-        if (compression)
+        if (compression && !buffer_span.empty())
         {
             return decompress(sparrow_ipc::details::from_fb_compression_type(compression->codec()), buffer_span);
         }