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

refs #2239

Change-Id: Ifcc03513b1cbe580de12725baa4338f822a71417
diff --git a/daemon/mgmt/face-status-publisher.cpp b/daemon/mgmt/face-status-publisher.cpp
index 04542c8..759f9ce 100644
--- a/daemon/mgmt/face-status-publisher.cpp
+++ b/daemon/mgmt/face-status-publisher.cpp
@@ -25,6 +25,7 @@
 #include "face-status-publisher.hpp"
 #include "core/logger.hpp"
 #include "fw/face-table.hpp"
+#include <boost/range/adaptor/reversed.hpp>
 
 #include <ndn-cxx/management/nfd-face-status.hpp>
 
@@ -54,9 +55,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) {
     ndn::nfd::FaceStatus status = face->getFaceStatus();
     totalLength += status.wireEncode(outBuffer);
   }