Mir

Merge lp:~afrantzis/mir/fix-1286010 into lp:mir

Proposed by Alexandros Frantzis
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 1459
Proposed branch: lp:~afrantzis/mir/fix-1286010
Merge into: lp:mir
Diff against target: 69 lines (+5/-3)
5 files modified
examples/image_renderer.cpp (+1/-0)
examples/render_surfaces.cpp (+0/-2)
include/test/mir_test_doubles/fake_renderable.h (+1/-0)
src/server/compositor/gl_renderer.cpp (+1/-0)
src/server/scene/basic_surface.cpp (+2/-1)
To merge this branch: bzr merge lp:~afrantzis/mir/fix-1286010
Reviewer Review Type Date Requested Status
Alan Griffiths Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+210185@code.launchpad.net

Commit message

Fix glm build warnings

Description of the change

Fix glm build warnings

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'examples/image_renderer.cpp'
2--- examples/image_renderer.cpp 2014-03-04 04:09:36 +0000
3+++ examples/image_renderer.cpp 2014-03-10 11:11:24 +0000
4@@ -23,6 +23,7 @@
5 #pragma GCC diagnostic warning "-Wall"
6 #include <glm/glm.hpp>
7 #pragma GCC diagnostic pop
8+#define GLM_FORCE_RADIANS
9 #include <glm/gtc/type_ptr.hpp>
10
11 #include <memory>
12
13=== modified file 'examples/render_surfaces.cpp'
14--- examples/render_surfaces.cpp 2014-03-06 19:05:47 +0000
15+++ examples/render_surfaces.cpp 2014-03-10 11:11:24 +0000
16@@ -49,8 +49,6 @@
17 #include <sstream>
18 #include <vector>
19
20-#include <glm/gtc/matrix_transform.hpp>
21-
22 namespace mg = mir::graphics;
23 namespace mc = mir::compositor;
24 namespace ms = mir::scene;
25
26=== modified file 'include/test/mir_test_doubles/fake_renderable.h'
27--- include/test/mir_test_doubles/fake_renderable.h 2014-03-06 05:29:00 +0000
28+++ include/test/mir_test_doubles/fake_renderable.h 2014-03-10 11:11:24 +0000
29@@ -20,6 +20,7 @@
30 #define MIR_TEST_DOUBLES_FAKE_RENDERABLE_H_
31
32 #include "mir/graphics/renderable.h"
33+#define GLM_FORCE_RADIANS
34 #include <glm/gtc/matrix_transform.hpp>
35 #include <gmock/gmock.h>
36
37
38=== modified file 'src/server/compositor/gl_renderer.cpp'
39--- src/server/compositor/gl_renderer.cpp 2014-03-04 04:09:36 +0000
40+++ src/server/compositor/gl_renderer.cpp 2014-03-10 11:11:24 +0000
41@@ -20,6 +20,7 @@
42 #include "mir/graphics/renderable.h"
43 #include "mir/graphics/buffer.h"
44
45+#define GLM_FORCE_RADIANS
46 #include <glm/gtc/matrix_transform.hpp>
47 #include <glm/gtc/type_ptr.hpp>
48
49
50=== modified file 'src/server/scene/basic_surface.cpp'
51--- src/server/scene/basic_surface.cpp 2014-03-06 19:05:47 +0000
52+++ src/server/scene/basic_surface.cpp 2014-03-10 11:11:24 +0000
53@@ -25,6 +25,7 @@
54
55 #include "mir/scene/scene_report.h"
56
57+#define GLM_FORCE_RADIANS
58 #include <glm/gtc/matrix_transform.hpp>
59
60 #include <boost/throw_exception.hpp>
61@@ -241,7 +242,7 @@
62 {
63 {
64 std::unique_lock<std::mutex> lk(guard);
65- rotation_matrix = glm::rotate(glm::mat4(1.0f), degrees, axis);
66+ rotation_matrix = glm::rotate(glm::mat4(1.0f), glm::radians(degrees), axis);
67 transformation_dirty = true;
68 }
69 notify_change();

Subscribers

People subscribed via source and target branches