Re: [boost] [graph][release][1.42] Debug code removal in release branch

Subject: Re: [boost] [graph][release][1.42] Debug code removal in release branch
From: Vladimir Prus (vladimir_at_[hidden])
Date: 2010-01-14 12:18:11

Jeremiah Willcock wrote:

> A user reported some debugging code that had been left in Boost.Graph for
> a couple of releases.  It is a simple patch and a really embarrassing bug
> to leave in.  It is change 59005 in trunk:
> 
> Index: read_graphviz_new.cpp
> ===================================================================
> --- read_graphviz_new.cpp	(revision 59004)
> +++ read_graphviz_new.cpp	(revision 59005)
> @@ -764,7 +764,7 @@
>       typedef boost::detail::graph::node_t vertex;
>       typedef boost::detail::graph::edge_t edge;
>       for (std::map<node_name, properties>::const_iterator i = r.nodes.begin(); i !=
>       r.nodes.end(); ++i) {
> -      std::cerr << i->first << " " << props_to_string(i->second) << std::endl;
> +      // std::cerr << i->first << " " << props_to_string(i->second) << std::endl;
>         mg->do_add_vertex(i->first);
>         for (properties::const_iterator j = i->second.begin(); j != i->second.end(); ++j) {
>           mg->set_node_property(j->first, i->first, j->second);
> @@ -772,7 +772,7 @@
>       }
>       for (std::vector<edge_info>::const_iterator i = r.edges.begin(); i != r.edges.end(); ++i) {
>         const edge_info& ei = *i;
> -      std::cerr << ei.source << " -> " << ei.target << " " << props_to_string(ei.props) <<
> std::endl;
> +      // std::cerr << ei.source << " -> " << ei.target << " " << props_to_string(ei.props) <<
> std::endl;
>         edge e = edge::new_edge();
>         mg->do_add_edge(e, ei.source.name, ei.target.name);
>         for (properties::const_iterator j = ei.props.begin(); j != ei.props.end(); ++j) {
> @@ -782,7 +782,7 @@
>       std::map<subgraph_name, properties>::const_iterator root_graph_props_i =
>       r.graph_props.find("___root___"); assert (root_graph_props_i != r.graph_props.end()); //
>       Should not happen const properties& root_graph_props = root_graph_props_i->second;
> -    std::cerr << "ending graph " << props_to_string(root_graph_props) << std::endl;
> +    // std::cerr << "ending graph " << props_to_string(root_graph_props) << std::endl;
>       for (properties::const_iterator i = root_graph_props.begin(); i != root_graph_props.end();
>       ++i) {
>         mg->set_graph_property(i->first, i->second);
>       }
> 
> May I please put this into the release branch for 1.42?  Thank you.

This change is fine for the release branch, thanks.

- Volodya