Merge branch 'unpacker-fix-counter' into 'master'

Unpacker: fix counter usage

See merge request !4147

Event/EventPacker/src/component/BufferUnpackerBaseAlg.h

@@ -70,6 +70,7 @@ namespace LHCb::Hlt::PackedData {
     Gaudi::Accumulators::MsgCounter<MSG::WARNING>     missing_dependency;
     Gaudi::Accumulators::BinomialCounter<>            nothing_to_do;
     Gaudi::Accumulators::SummingCounter<unsigned int> nbUnpackedData;
+    Gaudi::Accumulators::SummingCounter<unsigned int> sizeBuffer;
     Gaudi::Accumulators::SummingCounter<unsigned int> nbBufferData;
     Gaudi::Accumulators::SummingCounter<unsigned int> nbLinkedBufferData;
     Gaudi::Accumulators::SummingCounter<unsigned int> missingLinks;
@@ -86,6 +87,7 @@ namespace LHCb::Hlt::PackedData {
         , missing_dependency{parent, "DstData buffer for dependency not available - unable to unpack", 0}
         , nothing_to_do{parent, "Target data already present"}
         , nbUnpackedData{parent, "# UnpackedData"}
+        , sizeBuffer{parent, "Buffer size"}
         , nbBufferData{parent, "# BufferData"}
         , nbLinkedBufferData{parent, "# Linked BufferData"}
         , missingLinks{parent, "# Missing Link Locations"}
@@ -442,8 +444,7 @@ namespace DataPacking::Buffer {
       } else {
         // Count unpacked output
         m_counters.nbUnpackedData += size( **obj );
-        m_counters.nbUnpackedData +=
-            buffer->buffer().size(); // BUG: FIXME:  backwards bug-compatible. TODO: this must be a different counter..
+        m_counters.sizeBuffer += buffer->buffer().size();
 
         m_data.put( std::move( *obj ) );
       }