- Oct 07, 2015
-
-
- May 30, 2015
-
-
- Feb 20, 2014
-
-
Neil Roberts authored
The previous implementation was dereferencing the sample pointer in order to get the offset to subtract from the member pointer. The resulting value is then only used to get a pointer to the member in order to calculate the offset so it doesn't actually read from the memory location and shouldn't cause any problems. However this is probably technically invalid and could have undefined behaviour. It looks like clang takes advantage of this undefined behaviour and doesn't actually offset the pointer. It also generates a warning when it does this. This patch splits the _cogl_container_of macro into two implementations. Previously the macro was always used in the list iterator macros like this: SomeType *sample = _cogl_container_of(list_node, sample, link) Instead of doing that there is now a new macro called _cogl_list_set_iterator which explicitly assigns to the sample pointer with an initial value before assigning to it again with the real offset. This redundant initialisation gets optimised out by compiler. The second macro is still called _cogl_container_of but instead of taking a sample pointer it just directly takes the type name. That way it can use the standard offsetof macro. https://bugzilla.gnome.org/show_bug.cgi?id=723530 Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 1efed1e0)
-
- Jan 24, 2014
-
-
Wylmer Wang authored
-
- Jan 22, 2014
-
-
Lionel Landwerlin authored
Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 07114acc)
-
- Jan 20, 2014
-
-
Neil Roberts authored
-
Neil Roberts authored
-
Neil Roberts authored
-
Adel Gadllah authored
Commit a750f80c was supposed to enable it for newer mesa but was wrong, fix that. (cherry picked from commit 0548757e)
-
Adel Gadllah authored
The bug that prevented MESA_copy_sub_buffer to work for swrast / llvmpipe got fixed in mesa 10.1 git so enable it for mesa 10.1+. https://bugzilla.gnome.org/show_bug.cgi?id=721450 When landing the patch, it was tweaked to #include "cogl-version.h" to avoid a compiler warning about COGL_VERSION_ENCODE being implicitly defined. -- Robert Bragg Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit e7e216b1)
-
Neil Roberts authored
The -L option makes curl follow redirections. This is needed for downloading glext.h because khronos.org is using a redirect. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 85baaef4)
-
Neil Roberts authored
Since 248a76f5 cogl.h can no longer be included in internal source files so the WGL winsys was no longer compiling. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 91af97a2)
-
- Dec 17, 2013
-
-
Lionel Landwerlin authored
(cherry picked from commit 002e2ec0)
-
Lionel Landwerlin authored
(cherry picked from commit 637728dd)
-
Jasper St. Pierre authored
This means that we can't cache the journal read_pixels optimization. https://bugzilla.gnome.org/show_bug.cgi?id=719582 Reviewed-by: Neil Roberts <neil@linux.intel.com> Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 550bae22)
-
Neil Roberts authored
When projecting the bounding rectangle of a primitive it was using the modelview matrix twice instead of the modelview and projection matrices so it was coming out with garbage. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 7e1f05c8)
-
Neil Roberts authored
The test makes an L-shaped path that fills the whole framebuffer except for the top right quadrant. It then clips to that and tries to fill the framebuffer with a rectangle. Then it verifies that all of the quadrants have the expected colour. This is currently failing due to a bug in the primitive clipping. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 54040332)
-
Neil Roberts authored
(cherry picked from commit f2072ab0) Conflicts: .gitignore
-
Robert Bragg authored
This improves the error message in the case where libgbm is missing when the KMS egl platform has been enabled. https://bugzilla.gnome.org/show_bug.cgi?id=706808 Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 79ad4165)
-
- Oct 28, 2013
-
-
Robert Bragg authored
Depending on what version of Mesa you have then eglQueryWaylandBuffer may take a wl_buffer or wl_resource argument and the EGL header will only forward declare the corresponding type. The use of wl_buffer has been deprecated and so internally we assume that eglQueryWaylandBuffer takes a wl_resource but for compatibility we forward declare wl_resource in case we are building with EGL headers that still use wl_buffer. https://bugzilla.gnome.org/show_bug.cgi?id=710926 Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 9bd1ee54)
-
- Oct 24, 2013
-
-
Neil Roberts authored
wl_buffer has been deprecated in the server API and instead compositors should be directly passing the wl_resource pointer to eglQueryWaylandBuffer. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit f13278bc)
-
- Oct 15, 2013
-
-
Emilio Pozuelo Monfort authored
Commit 50d1285b updated the wl_shm enum values but left one out. Update it to fix the build on big-endian. https://bugzilla.gnome.org/show_bug.cgi?id=710135 Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 92462868)
-
Chun-wei Fan authored
cogl_is_atlas_texture is supposed to be exported from the DLL/.so, so update the cogl.symbols file to ensure this. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit 13e037f0)
-
Robert Bragg authored
This makes cogl_framebuffer_set_color_mask immediately bail out if the given mask equals the framebuffer's current mask, since the cost of flushing the journal and flushing the gl state will hugely outweigh the cost of the check. Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 925174d9)
-
Robert Bragg authored
This avoids including wayland-server.h in cogl-display-private.h which avoid lots of compile time warnings that wl_buffer is deprecated. The problem is that wl_buffer is also exposed in the client side headers and isn't deprecated for clients. If we end up including the client and server headers in the same compilation unit we can get conflicting definitions. Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 60fcd9c8)
-
Robert Bragg authored
This updates the definition of _COGL_STATIC_ASSERT to just use _Static_assert if available or be NOP if not. We no longer worry about supporting static assertions with older compilers. This fixes some verbose warnings that newer compilers were giving with the old typedef based static assertion method. Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 645e3607)
-
Robert Bragg authored
When a pipeline is notified of a change we now make sure to notify all progends of that change not just the progend directly associated with that pipeline. A pipeline can have private state associated with it from multiple progends because descendants will always try and cache state on ancestors to maximize the chance that the state can later be re-used. Descendants may be using different progends than the ancestors that they cache state with. Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit 873939a1)
-
Robert Bragg authored
In the cases where we cache vertex state with an ancestor pipeline (whose vertex processing state is equivalent) we need to invalidate that state if that ancestor is later modified. This conformance test checks this case but currently fails because we only notify the progend directly associated with the pipeline being changed. In this case the pipeline can be using a different progend to the ancestor which it is caching state with so when the ancestor is changed it needs to notify all the progends that they may need to clear their private state. Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit e112af5f)
-
- Sep 23, 2013
-
-
Robert Bragg authored
Commit 7b25c8f5 mistakenly added a reference to a COGL_GTYPE_DEFINE_CLASS() macro that does not yet exist so this patch removes it. Reviewed-by: Neil Roberts <neil@linux.intel.com>
-
Neil Roberts authored
Previously when we released a .bzip2 file the Gnome release scripts will just convert this to a .xz file and generate its own checksum. The .bzip2 is never actually available on the FTP server. This isn't ideal because we were listing the checksum of the .bzip2 file in the release message but that is useless information. This patch changes it to generate a .xz file instead which is also what Clutter does. Reviewed-by: Robert Bragg <robert@linux.intel.com> (cherry picked from commit cd20c869)
-
Robert Bragg authored
-
Robert Bragg authored
-
Robert Bragg authored
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-
Lionel Landwerlin authored
Reviewed-by: Robert Bragg <robert@linux.intel.com>
-