Merge lp:~teemperor/pantheon-greeter/without-gl into lp:~elementary-pantheon/pantheon-greeter/trunk

Proposed by Raphael Isemann
Status: Rejected
Rejected by: Cody Garver
Proposed branch: lp:~teemperor/pantheon-greeter/without-gl
Merge into: lp:~elementary-pantheon/pantheon-greeter/trunk
Diff against target: 68 lines (+1/-27)
3 files modified
CMakeLists.txt (+0/-1)
src/Wallpaper.vala (+1/-9)
vapi/gl.vapi (+0/-17)
To merge this branch: bzr merge lp:~teemperor/pantheon-greeter/without-gl
Reviewer Review Type Date Requested Status
Cody Garver (community) Needs Information
Review via email: mp+192783@code.launchpad.net

Description of the change

I think we can merge this branch without issues. Please test some bigger wallpaper with this branch.

To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) wrote :

Wasn't there a bug involving gl where if the wallpaper was too large, it made a black background? I seem to remember gl being the fix for that. If that's accurate, how would this handle a huge wallpaper?

review: Needs Information
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Either experimental or very latest Clutter can tile textures to be able to process images bigger than the hardware allows natively. However, last time I checked this feature was under experimental API, which means that you might need to change some Clutter API calls.

If such functionality in Clutter still isn't stable and is not going to be stable by 14.04, we're probably better off with developing an OpenGL ES codepath equivalent to the current OpenGL one.

Unmerged revisions

214. By Raphael Isemann

removed gl dep for arm-testing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2013-10-08 18:27:22 +0000
+++ CMakeLists.txt 2013-10-26 15:32:36 +0000
@@ -51,7 +51,6 @@
51 x1151 x11
52 clutter-gtk-1.052 clutter-gtk-1.0
53 indicator-0.453 indicator-0.4
54 gl
55 posix-fixes54 posix-fixes
56OPTIONS55OPTIONS
57 --vapidir=${CMAKE_CURRENT_SOURCE_DIR}/vapi56 --vapidir=${CMAKE_CURRENT_SOURCE_DIR}/vapi
5857
=== modified file 'src/Wallpaper.vala'
--- src/Wallpaper.vala 2013-10-08 18:16:40 +0000
+++ src/Wallpaper.vala 2013-10-26 15:32:36 +0000
@@ -25,8 +25,6 @@
2525
26 bool second = false;26 bool second = false;
2727
28 int gpu_limit;
29
30 string[] cache_path = {};28 string[] cache_path = {};
31 Gdk.Pixbuf[] cache_pixbuf = {};29 Gdk.Pixbuf[] cache_pixbuf = {};
32 int max_cache = 3;30 int max_cache = 3;
@@ -46,10 +44,6 @@
4644
47 add_child (background);45 add_child (background);
48 add_child (background_s);46 add_child (background_s);
49
50 GL.GLint result = 1;
51 GL.glGetIntegerv(GL.GL_MAX_TEXTURE_SIZE, out result);
52 gpu_limit = result;
53 }47 }
5448
55 string get_default () {49 string get_default () {
@@ -149,9 +143,7 @@
149 * screen size to save memory.143 * screen size to save memory.
150 */144 */
151 public Gdk.Pixbuf validate_pixbuf (Gdk.Pixbuf pixbuf) {145 public Gdk.Pixbuf validate_pixbuf (Gdk.Pixbuf pixbuf) {
152 Gdk.Pixbuf result = scale_to_rect (pixbuf, gpu_limit, gpu_limit);146 return pixbuf;
153 result = scale_to_rect (pixbuf, screen_width, screen_height);
154 return result;
155 }147 }
156148
157 /**149 /**
158150
=== removed file 'vapi/gl.vapi'
--- vapi/gl.vapi 2013-02-17 13:12:35 +0000
+++ vapi/gl.vapi 1970-01-01 00:00:00 +0000
@@ -1,17 +0,0 @@
1/* gl.vapi generated by vapigen, do not modify. */
2
3[CCode (cprefix = "GL", gir_namespace = "GL", gir_version = "1.0", lower_case_cprefix = "gl_")]
4namespace GL {
5 [CCode (cheader_filename = "GL/gl.h", cname = "GLint")]
6 [SimpleType]
7 public struct GLint : int {
8 }
9 [CCode (cheader_filename = "GL/gl.h", cname = "GLenum")]
10 [SimpleType]
11 public struct GLenum : uint {
12 }
13 [CCode (cheader_filename = "GL/gl.h", cname = "GL_MAX_TEXTURE_SIZE")]
14 public const int GL_MAX_TEXTURE_SIZE;
15 [CCode (cheader_filename = "GL/gl.h", cname = "glGetIntegerv")]
16 public static void glGetIntegerv (GL.GLenum pname, out unowned GL.GLint @params);
17}

Subscribers

People subscribed via source and target branches