commit a3d9fc08254d7be5508523f93a9e28906813ee8f
author Eric Newberry <enewberry@email.arizona.edu> Sun Jul 26 10:55:44 2015 -0700
committer Eric Newberry <enewberry@email.arizona.edu> Sun Jul 26 20:14:32 2015 -0700
tree 2da6e33de81d9fb96b077bb303a681316d4368ce
parent 2096e18e352b2d8c743eeddce03779a1a23d698a

encoding: Block::remove now removes all subelements of type

refs #3070

Change-Id: Ife9cb9fe9ba7e0a79d5f759c9990cbe04b0b87c3

src/encoding/block.cpp

diff --git a/src/encoding/block.cpp b/src/encoding/block.cpp
index 693bc17..77be4f2 100644
--- a/src/encoding/block.cpp
+++ b/src/encoding/block.cpp
@@ -406,7 +406,7 @@
   resetWire();
 
   auto it = std::remove_if(m_subBlocks.begin(), m_subBlocks.end(),
-                           [type] (const Block& subBlock) { return subBlock.type() != type; });
+                           [type] (const Block& subBlock) { return subBlock.type() == type; });
   m_subBlocks.resize(it - m_subBlocks.begin());
 }
 

src/encoding/block.hpp

diff --git a/src/encoding/block.hpp b/src/encoding/block.hpp
index 9407ccf..0552755 100644
--- a/src/encoding/block.hpp
+++ b/src/encoding/block.hpp
@@ -249,6 +249,11 @@
   element_const_iterator
   find(uint32_t type) const;
 
+  /**
+   * @brief remove all subelements of \p type
+   * @param type TLV-TYPE of subelements to remove
+   * @pre parse() has been invoked
+   */
   void
   remove(uint32_t type);