Fix: Minor changes

Author: harrysroberts

src/dodgr.dll

@@ -192,7 +192,6 @@ size_t graph::identify_graph_components (vertex_map_t &v,
 //' @param index used to execute algoritm
 //' @param compnum the current component number
 //' @noRd
-
 void graph::strong_connect(vertex_id_t vt,
     vertex_map_t &v,
     std::unordered_map <vertex_id_t, size_t> &com,
@@ -299,15 +298,14 @@ size_t graph::identify_graph_strong_components (vertex_map_t &v,
 //' numbers
 //' @noRd
 // [[Rcpp::export]]
-
 Rcpp::List rcpp_get_component_vector (const Rcpp::DataFrame &graph, bool strong = false)
 {
     vertex_map_t vertices;
     edge_map_t edge_map;
     vert2edge_map_t vert2edge_map;
 
     bool has_times = graph::graph_from_df (graph, vertices, edge_map, vert2edge_map);
-    has_times = false; // suppress unused variable warning
+    has_times = false; // rm unused variable warning
 
     std::unordered_map <vertex_id_t, size_t> components;
     size_t largest_component;
@@ -319,9 +317,9 @@ Rcpp::List rcpp_get_component_vector (const Rcpp::DataFrame &graph, bool strong
 
     largest_component++; // suppress unused variable warning
 
-    // Map component numbers of vertices onto edges
+    // Then map component numbers of vertices onto edges
     std::unordered_map <edge_id_t, size_t> comp_nums;
-    for (auto &e : edge_map)
+    for (auto e : edge_map)
     {
         vertex_id_t vi = e.second.get_from_vertex ();
         comp_nums.emplace (e.first, components.find(vi)->second);
@@ -330,7 +328,7 @@ Rcpp::List rcpp_get_component_vector (const Rcpp::DataFrame &graph, bool strong
     Rcpp::StringVector edge_id (comp_nums.size ());
     Rcpp::IntegerVector comp_num (comp_nums.size ());
     size_t i = 0;
-    for (auto &cn: comp_nums)
+    for (auto cn: comp_nums)
     {
         edge_id (i) = cn.first;
         comp_num (i) = static_cast <int> (cn.second) + 1L; // 1-indexed for R