core: eliminate MapValueIterator in favor of boost::adaptors::map_values

refs #2239

Change-Id: Ifcc03513b1cbe580de12725baa4338f822a71417
diff --git a/daemon/mgmt/face-query-status-publisher.cpp b/daemon/mgmt/face-query-status-publisher.cpp
index 02fd4b6..d208087 100644
--- a/daemon/mgmt/face-query-status-publisher.cpp
+++ b/daemon/mgmt/face-query-status-publisher.cpp
@@ -25,6 +25,7 @@
 
 #include "face-query-status-publisher.hpp"
 #include "core/logger.hpp"
+#include <boost/range/adaptor/reversed.hpp>
 
 #include <ndn-cxx/management/nfd-face-status.hpp>
 
@@ -95,10 +96,7 @@
 {
   size_t totalLength = 0;
 
-  for (FaceTable::const_reverse_iterator i = m_faceTable.rbegin();
-       i != m_faceTable.rend(); ++i) {
-    const shared_ptr<Face>& face = *i;
-
+  for (const shared_ptr<Face>& face : m_faceTable | boost::adaptors::reversed) {
     if (doesMatchFilter(face)) {
       ndn::nfd::FaceStatus status = face->getFaceStatus();
       totalLength += status.wireEncode(outBuffer);