Changeset 692 for trunk/src/cairo_renderer.cpp

Timestamp:

04/11/08 19:11:35 (9 months ago)

Author:

tom

Message:

Don't try and render text if the attribute containing the text does
not exist for the feature. Closes #89.

Files:

• trunk/src/cairo_renderer.cpp (modified) (2 diffs)

trunk/src/cairo_renderer.cpp

@@ -751 +751 @@
       typedef coord_transform2<CoordTransform,geometry2d> path_type;
 
-      UnicodeString text = feature[sym.get_name()].to_unicode();
-      boost::shared_ptr<ImageData32> const& data = sym.get_image();
-
-      if (text.length() > 0 && data)
-      {
-         cairo_face_ptr face = face_manager_.get_face(sym.get_face_name());
-
-         if (face)
-         {
-            cairo_context context(context_);
-
-            context.set_font_face(face);
-
-            string_info info(text);
-
-            face->get_string_info(sym.get_text_size(), info);
-
-            placement_finder<label_collision_detector4> finder(detector_);
-
-            for (unsigned i = 0; i < feature.num_geometries(); ++i)
-            {
-               geometry2d const& geom = feature.get_geometry(i);
-
-               if (geom.num_points() > 0) // don't bother with empty geometries
-               {
-                  path_type path(t_, geom, prj_trans);
-                  placement text_placement(info, sym);
-
-                  text_placement.avoid_edges = sym.get_avoid_edges();
-                  finder.find_point_placements<path_type>(text_placement, path);
-
-                  for (unsigned int ii = 0; ii < text_placement.placements.size(); ++ ii)
+      if (feature.exists(sym.get_name()))
+      {
+         UnicodeString text = feature[sym.get_name()].to_unicode();
+         boost::shared_ptr<ImageData32> const& data = sym.get_image();
+
+         if (text.length() > 0 && data)
+         {
+            cairo_face_ptr face = face_manager_.get_face(sym.get_face_name());
+
+            if (face)
+            {
+               cairo_context context(context_);
+
+               context.set_font_face(face);
+
+               string_info info(text);
+
+               face->get_string_info(sym.get_text_size(), info);
+
+               placement_finder<label_collision_detector4> finder(detector_);
+
+               for (unsigned i = 0; i < feature.num_geometries(); ++i)
+               {
+                  geometry2d const& geom = feature.get_geometry(i);
+
+                  if (geom.num_points() > 0) // don't bother with empty geometries
                   {
-                     double shield_x = text_placement.placements[ii].starting_x - data->width() / 2;
-                     double shield_y = text_placement.placements[ii].starting_y - data->height() / 2;
-
-                     context.add_image(shield_x, shield_y, *data);
-                     context.add_text(sym, text_placement.placements[ii], face);
+                     path_type path(t_, geom, prj_trans);
+                     placement text_placement(info, sym);
+
+                     text_placement.avoid_edges = sym.get_avoid_edges();
+                     finder.find_point_placements<path_type>(text_placement, path);
+
+                     for (unsigned int ii = 0; ii < text_placement.placements.size(); ++ ii)
+                     {
+                        double shield_x = text_placement.placements[ii].starting_x - data->width() / 2;
+                        double shield_y = text_placement.placements[ii].starting_y - data->height() / 2;
+
+                        context.add_image(shield_x, shield_y, *data);
+                        context.add_text(sym, text_placement.placements[ii], face);
+                     }
                   }
                }
@@ -925 +928 @@
       typedef coord_transform2<CoordTransform,geometry2d> path_type;
 
-      UnicodeString text = feature[sym.get_name()].to_unicode();
-
-      if (text.length() > 0)
-      {
-         cairo_face_ptr face = face_manager_.get_face(sym.get_face_name());
-
-         if (face)
-         {
-            cairo_context context(context_);
-
-            context.set_font_face(face);
-
-            string_info info(text);
-
-            face->get_string_info(sym.get_text_size(), info);
-
-            placement_finder<label_collision_detector4> finder(detector_);
-
-            for (unsigned i = 0; i < feature.num_geometries(); ++i)
-            {
-               geometry2d const& geom = feature.get_geometry(i);
-
-               if (geom.num_points() > 0) // don't bother with empty geometries
-               {
-                  path_type path(t_, geom, prj_trans);
-                  placement text_placement(info, sym);
-
-                  if (sym.get_label_placement() == POINT_PLACEMENT)
+      if (feature.exists(sym.get_name()))
+      {
+         UnicodeString text = feature[sym.get_name()].to_unicode();
+
+         if (text.length() > 0)
+         {
+            cairo_face_ptr face = face_manager_.get_face(sym.get_face_name());
+
+            if (face)
+            {
+               cairo_context context(context_);
+
+               context.set_font_face(face);
+
+               string_info info(text);
+
+               face->get_string_info(sym.get_text_size(), info);
+
+               placement_finder<label_collision_detector4> finder(detector_);
+
+               for (unsigned i = 0; i < feature.num_geometries(); ++i)
+               {
+                  geometry2d const& geom = feature.get_geometry(i);
+
+                  if (geom.num_points() > 0) // don't bother with empty geometries
                   {
-                     double label_x, label_y, z = 0.0;
-
-                     geom.label_position(&label_x, &label_y);
-                     prj_trans.backward(label_x, label_y, z);
-                     t_.forward(&label_x, &label_y);
-                     finder.find_point_placement(text_placement, label_x, label_y);
+                     path_type path(t_, geom, prj_trans);
+                     placement text_placement(info, sym);
+
+                     if (sym.get_label_placement() == POINT_PLACEMENT)
+                     {
+                        double label_x, label_y, z = 0.0;
+
+                        geom.label_position(&label_x, &label_y);
+                        prj_trans.backward(label_x, label_y, z);
+                        t_.forward(&label_x, &label_y);
+                        finder.find_point_placement(text_placement, label_x, label_y);
+                     }
+                     else //LINE_PLACEMENT
+                     {
+                        finder.find_line_placements<path_type>(text_placement, path);
+                     }
+
+                     for (unsigned int ii = 0; ii < text_placement.placements.size(); ++ii)
+                     {
+                        context.add_text(sym, text_placement.placements[ii], face);
+                     }
                   }
-                  else //LINE_PLACEMENT
-                  {
-                     finder.find_line_placements<path_type>(text_placement, path);
-                  }
-
-                  for (unsigned int ii = 0; ii < text_placement.placements.size(); ++ii)
-                  {
-                     context.add_text(sym, text_placement.placements[ii], face);
-                  }
-               }
-            }
-         }
-         else
-         {
-            throw config_error("Unable to find specified font face '" + sym.get_face_name() + "'");
+               }
+            }
+            else
+            {
+               throw config_error("Unable to find specified font face '" + sym.get_face_name() + "'");
+            }
          }
       }