Merge lp:~nha/widelands/bug1125539 into lp:widelands

Proposed by Nicolai Hähnle
Status: Merged
Merged at revision: 6514
Proposed branch: lp:~nha/widelands/bug1125539
Merge into: lp:widelands
Diff against target: 44 lines (+8/-4)
2 files modified
src/graphic/render/gamerenderer_gl.cc (+6/-3)
src/graphic/render/gamerenderer_sdl.cc (+2/-1)
To merge this branch: bzr merge lp:~nha/widelands/bug1125539
Reviewer Review Type Date Requested Status
SirVer Approve
Review via email: mp+149272@code.launchpad.net

Commit message

Fix for bug #1125539: Always render the road overlay, even when the player sees all

To post a comment you must log in.
Revision history for this message
SirVer (sirver) wrote :

lgtm.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/graphic/render/gamerenderer_gl.cc'
2--- src/graphic/render/gamerenderer_gl.cc 2013-02-11 18:01:26 +0000
3+++ src/graphic/render/gamerenderer_gl.cc 2013-02-19 13:25:32 +0000
4@@ -501,13 +501,16 @@
5
6 uint8_t GameRendererGL::field_roads(const FCoords & coords) const
7 {
8+ uint8_t roads;
9+ const Map & map = m_egbase->map();
10 if (m_player && !m_player->see_all()) {
11- const Map & map = m_egbase->map();
12 const Player::Field & pf = m_player->fields()[Map::get_index(coords, map.get_width())];
13- return pf.roads | map.overlay_manager().get_road_overlay(coords);
14+ roads = pf.roads | map.overlay_manager().get_road_overlay(coords);
15 } else {
16- return coords.field->get_roads();
17+ roads = coords.field->get_roads();
18 }
19+ roads |= map.overlay_manager().get_road_overlay(coords);
20+ return roads;
21 }
22
23 void GameRendererGL::prepare_roads()
24
25=== modified file 'src/graphic/render/gamerenderer_sdl.cc'
26--- src/graphic/render/gamerenderer_sdl.cc 2013-02-10 16:41:12 +0000
27+++ src/graphic/render/gamerenderer_sdl.cc 2013-02-19 13:25:32 +0000
28@@ -151,7 +151,7 @@
29 f_d_texture = get_terrain_texture(f_pf.terrains.d);
30 tr_d_texture = get_terrain_texture(tr_pf.terrains.d);
31
32- roads = f_pf.roads | map.overlay_manager().get_road_overlay(f);
33+ roads = f_pf.roads;
34
35 f_brightness = node_brightness
36 (gametime, f_pf.time_node_last_unseen,
37@@ -177,6 +177,7 @@
38 bl_brightness = bl.field->get_brightness();
39 br_brightness = br.field->get_brightness();
40 }
41+ roads |= map.overlay_manager().get_road_overlay(f);
42
43 Vertex f_vert
44 (f_posx, posy - f.field->get_height() * HEIGHT_FACTOR,

Subscribers

People subscribed via source and target branches

to status/vote changes: