sij/mapnik
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix uneeded pointer indirection with glyph_info/glyph_position

Browse source
This commit is contained in:
Dane Springmeyer 2014-10-09 18:38:29 -07:00
parent aed5243302
commit 536b23bfde
4 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
include/mapnik/text/placements_list.hpp
View file

@ -42,8 +42,8 @@ struct glyph_info;
struct glyph_position
{
glyph_position(glyph_info const& _glyph, pixel_position const& _pos, rotation const& _rot)
: glyph(&_glyph), pos(_pos), rot(_rot) { }
glyph_info const* glyph;
: glyph(_glyph), pos(_pos), rot(_rot) { }
glyph_info const& glyph;
pixel_position pos;
rotation rot;
};

6
src/cairo/cairo_context.cpp
View file

@ -447,7 +447,7 @@ void cairo_context::add_text(glyph_positions_ptr path,
set_operator(halo_comp_op);
for (auto const& glyph_pos : *path)
{
glyph_info const& glyph = *(glyph_pos.glyph);
glyph_info const& glyph = glyph_pos.glyph;
if (glyph.format)
{
@ -479,9 +479,9 @@ void cairo_context::add_text(glyph_positions_ptr path,
stroke();
}
set_operator(comp_op);
for (auto const &glyph_pos : *path)
for (auto const& glyph_pos : *path)
{
glyph_info const& glyph = *(glyph_pos.glyph);
glyph_info const& glyph = glyph_pos.glyph;
if (glyph.format)
{

2
src/renderer_common/process_group_symbolizer.cpp
View file

@ -64,7 +64,7 @@ text_render_thunk::text_render_thunk(placements_list const& placements,
for (glyph_position const& pos : *positions)
{
glyph_vec.push_back(*pos.glyph);
glyph_vec.push_back(pos.glyph);
new_pos.push_back(glyph_vec.back(), pos.pos, pos.rot);
}

2
src/text/renderer.cpp
View file

@ -61,7 +61,7 @@ void text_renderer::prepare_glyphs(glyph_positions const& positions)
for (auto const& glyph_pos : positions)
{
glyph_info const& glyph = *(glyph_pos.glyph);
glyph_info const& glyph = glyph_pos.glyph;
glyph.face->set_character_sizes(glyph.format->text_size * scale_factor_); //TODO: Optimize this?
matrix.xx = static_cast<FT_Fixed>( glyph_pos.rot.cos * 0x10000L);