Changeset 62:80eefca04b1e in glemon-0.x

Timestamp:

08/18/05 15:33:49 (19 years ago)

Author:

Alpar Juttner

Branch:

gui

Phase:

public

Convert:

svn:c9d7d8f5-90d6-0310-b91f-818b3a526b0e/lemon/trunk/gui@2156

Message:

• The number of gcc-4.0 warnings has significantly decreases.
• Some code clean-up in gui

Files:

• all_include.h (modified) (1 diff)
• graph_displayer_canvas-edge.cc (modified) (3 diffs)
• graph_displayer_canvas-event.cc (modified) (8 diffs)
• graph_displayer_canvas-node.cc (modified) (3 diffs)
• graph_displayer_canvas.h (modified) (3 diffs)
• map_win.cc (modified) (2 diffs)
• mapstorage.cc (modified) (1 diff)

all_include.h

@@ -41 +41 @@
 typedef Graph::NodeMap<Coordinates> CoordinatesMap;
 typedef Graph::Node Node;
+typedef Graph::Edge Edge;
 typedef Graph::EdgeIt EdgeIt;
 typedef Graph::NodeIt NodeIt;

graph_displayer_canvas-edge.cc

@@ -4 +4 @@
 
 
-int GraphDisplayerCanvas::changeEdgeWidth (std::string mapname, Graph::Edge edge)
+int GraphDisplayerCanvas::changeEdgeWidth (std::string mapname, Edge edge)
 {
   Graph::EdgeMap<double> * actual_map;
@@ -50 +50 @@
 };
 
-int GraphDisplayerCanvas::changeEdgeColor (std::string mapname, Graph::Edge edge)
+int GraphDisplayerCanvas::changeEdgeColor (std::string mapname, Edge edge)
 {  
 
@@ -117 +117 @@
 };
 
-int GraphDisplayerCanvas::changeEdgeText (std::string mapname, Graph::Edge edge)
+int GraphDisplayerCanvas::changeEdgeText (std::string mapname, Edge edge)
 {
   //the number in the map will be written on the edge

graph_displayer_canvas-event.cc

@@ -226 +226 @@
       isbutton=1;
 
-      active_node=NodeIt(mapstorage.graph,mapstorage.graph.addNode());
+      active_node=mapstorage.graph.addNode();
 
       //initiating values corresponding to new node in maps
@@ -340 +340 @@
               window_to_world (e->button.x, e->button.y, clicked_x, clicked_y);
               target_item=(get_item_at(clicked_x, clicked_y));
-              Graph::NodeIt target_node=INVALID;
+              Node target_node=INVALID;
               for (NodeIt i(mapstorage.graph); i!=INVALID; ++i)
                 {
@@ -355 +355 @@
           mapstorage.modified = true;
 
-                      *(nodesmap[target_node]) << Gnome::Canvas::Properties::fill_color("red");
+                      *(nodesmap[target_node]) <<
+                        Gnome::Canvas::Properties::fill_color("red");
 
                       //creating new edge
-                      active_edge=EdgeIt(mapstorage.graph,mapstorage.graph.addEdge(active_node, target_node));
+                      active_edge=mapstorage.graph.addEdge(active_node,
+                                                           target_node);
 
                       //initiating values corresponding to new edge in maps
                       mapstorage.initMapsForEdge(active_edge);
-                      (*mapstorage.edgemap_storage["id"])[active_edge] = mapstorage.graph.id(active_edge);
+                      (*mapstorage.edgemap_storage["id"])[active_edge] = 
+                        mapstorage.graph.id(active_edge);
           
                       //calculating coordinates of new edge
@@ -556 +559 @@
           {
             //for determine, whether it was an edge
-            Graph::EdgeIt clicked_edge=INVALID;
+            Edge clicked_edge=INVALID;
 
             //find the activated item between texts
@@ -678 +681 @@
           {
             //for determine, whether it was a node
-            Graph::NodeIt clicked_node=INVALID;
+            Node clicked_node=INVALID;
 
             //find the activated item between texts
@@ -850 +853 @@
 }
 
-void GraphDisplayerCanvas::deleteItem(NodeIt node_to_delete)
+void GraphDisplayerCanvas::deleteItem(Node node_to_delete)
 {
   delete(nodetextmap[node_to_delete]);
@@ -857 +860 @@
 }
 
-void GraphDisplayerCanvas::deleteItem(EdgeIt edge_to_delete)
+void GraphDisplayerCanvas::deleteItem(Edge edge_to_delete)
 {
   delete(edgetextmap[edge_to_delete]);
@@ -864 +867 @@
 }
 
-void GraphDisplayerCanvas::deleteItem(Graph::Edge edge_to_delete)
-{
-  delete(edgetextmap[edge_to_delete]);
-  delete(edgesmap[edge_to_delete]);
-  mapstorage.graph.erase(edge_to_delete);
-}
-
 void GraphDisplayerCanvas::textReposition(xy<double> new_place)
 {

graph_displayer_canvas-node.cc

@@ -4 +4 @@
 
 
-int GraphDisplayerCanvas::changeNodeRadius (std::string mapname, Graph::Node node)
+int GraphDisplayerCanvas::changeNodeRadius (std::string mapname, Node node)
 {
   Graph::NodeMap<double> * actual_map;
@@ -70 +70 @@
 };
 
-int GraphDisplayerCanvas::changeNodeColor (std::string mapname, Graph::Node node)
+int GraphDisplayerCanvas::changeNodeColor (std::string mapname, Node node)
 {  
 
@@ -142 +142 @@
 };
 
-int GraphDisplayerCanvas::changeNodeText (std::string mapname, Graph::Node node)
+int GraphDisplayerCanvas::changeNodeText (std::string mapname, Node node)
 {
 

graph_displayer_canvas.h

@@ -24 +24 @@
   ///Changes the linewidth attribute according to the given map.
   ///\param mapname is the name of the map which contains the new values
-  int changeEdgeWidth (std::string mapname, Graph::Edge new_item=INVALID);
+  int changeEdgeWidth (std::string mapname, Edge new_item=INVALID);
 
   ///Changes the linecolor attribute according to the given map.
   ///\param mapname is the name of the map which contains the new values
-  int changeEdgeColor (std::string mapname, Graph::Edge new_item=INVALID);
+  int changeEdgeColor (std::string mapname, Edge new_item=INVALID);
 
   ///Changes the text of line attribute according to the given map.
   ///\param mapname is the name of the map which contains the new values
-  int changeEdgeText (std::string mapname, Graph::Edge new_item=INVALID);
+  int changeEdgeText (std::string mapname, Edge new_item=INVALID);
 
   ///Changes the linewidth attribute according to the given map.
   ///\param mapname is the name of the map which contains the new values
-  int changeNodeRadius (std::string mapname, Graph::Node new_item=INVALID);
+  int changeNodeRadius (std::string mapname, Node new_item=INVALID);
 
   ///Changes the linecolor attribute according to the given map.
   ///\param mapname is the name of the map which contains the new values
-  int changeNodeColor (std::string mapname, Graph::Node new_item=INVALID);
+  int changeNodeColor (std::string mapname, Node new_item=INVALID);
 
   ///Changes the text of line attribute according to the given map.
   ///\param mapname is the name of the map which contains the new values
-  int changeNodeText (std::string mapname, Graph::Node new_item=INVALID);
+  int changeNodeText (std::string mapname, Node new_item=INVALID);
 
   ///Callback for 'ViewZoomIn' action.
@@ -120 +120 @@
 private:
   ///Deletes the given element.
-  void deleteItem(NodeIt);
+  void deleteItem(Node);
   ///Deletes the given element.
-  void deleteItem(EdgeIt);
-  ///Deletes the given element.
-  void deleteItem(Graph::Edge);
+  void deleteItem(Edge);
 
 private:
@@ -167 +165 @@
   ///2. we would like to handle only ony item per movement, therefore quering it is not a working solution
   Gnome::Canvas::Item * active_item, * target_item;
-  Graph::NodeIt active_node;
-  Graph::EdgeIt active_edge;
-  Graph::EdgeIt forming_edge;
+  Node active_node;
+  Edge active_edge;
+  Edge forming_edge;
 
   std::string nodemap_to_edit, edgemap_to_edit;

map_win.cc

@@ -218 +218 @@
 };
 
-void MapWin::updateNode(Graph::Node node)
+void MapWin::updateNode(Node node)
 {
   for(int i=0;i<NODE_PROPERTY_NUM;i++)
@@ -251 +251 @@
 }
 
-void MapWin::updateEdge(Graph::Edge edge)
+void MapWin::updateEdge(Edge edge)
 {
   for(int i=0;i<EDGE_PROPERTY_NUM;i++)

mapstorage.cc

@@ -117 +117 @@
 }
 
-void MapStorage::initMapsForEdge(Graph::Edge e)
+void MapStorage::initMapsForEdge(Edge e)
 {
   std::map< std::string,Graph::EdgeMap<double> * >::iterator ems_it;