unit: clean up getValidFdrEngines()

diff --git a/unit/internal/fdr.cpp b/unit/internal/fdr.cpp
index d10c25f0cb61658985f7cd2be995b0ebd96aec98..ddfa8bb15d097007aba20284eb9d49f22efaa449 100644 (file)
--- a/unit/internal/fdr.cpp
+++ b/unit/internal/fdr.cpp
@@ -133,23 +133,25 @@ hwlmcb_rv_t decentCallbackT(size_t start, size_t end, u32 id, void *ctxt) {
 
 } // namespace
 
-static vector<u32> getValidFdrEngines() {
+static
+vector<u32> getValidFdrEngines() {
+    const auto target = get_current_target();
+
     vector<u32> ret;
-    vector<FDREngineDescription> des;
-    getFdrDescriptions(&des);
-    for (vector<FDREngineDescription>::const_iterator it = des.begin();
-         it != des.end(); ++it) {
-        if (it->isValidOnTarget(get_current_target())) {
-            ret.push_back(it->getID());
+
+    vector<FDREngineDescription> fdr_descriptions;
+    getFdrDescriptions(&fdr_descriptions);
+    for (const FDREngineDescription &d : fdr_descriptions) {
+        if (d.isValidOnTarget(target)) {
+            ret.push_back(d.getID());
         }
     }
 
-    vector<TeddyEngineDescription> tDes;
-    getTeddyDescriptions(&tDes);
-    for (vector<TeddyEngineDescription>::const_iterator it = tDes.begin();
-         it != tDes.end(); ++it) {
-        if (it->isValidOnTarget(get_current_target())) {
-            ret.push_back(it->getID());
+    vector<TeddyEngineDescription> teddy_descriptions;
+    getTeddyDescriptions(&teddy_descriptions);
+    for (const TeddyEngineDescription &d : teddy_descriptions) {
+        if (d.isValidOnTarget(target)) {
+            ret.push_back(d.getID());
         }
     }