~vcs-imports/libpixman/master : changes

~vcs-imports/libpixman/master » Changes from revision 2164

From Revision 2164 to 2145

Rev	Summary	Authors	Date
2164	configure: replace bugzilla URL with gitlab issues configure: replace bugzilla URL with gitlab issues Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>	Alan Coopersmith	2 years ago
2163	meson: Fix usage of pkgconfig.generate() meson: Fix usage of pkgconfig.generate() The library that the pkgconfig file is for should be the first positional argument. The `libraries:` kwarg is for libraries that the user must also link against, and which meson does not know about (and hence cannot automatically add to the `Libs:` or `Requires:` section in the .pc file). Fixes: ``` subprojects/pixman/meson.build:564: DEPRECATION: Library pixman-1 was passed to the "libraries" keyword argument of a previous call to generate() method instead of first positional argument. Adding pixman-1 to "Requires" field, but this is a deprecated behaviour that will change in a future version of Meson. Please report the issue if this warning cannot be avoided in your case. ```	Nirbheek Chauhan	2 years ago
2162	meson: Fix warning about extract_all_objects usage meson: Fix warning about extract_all_objects usage We use this because of a meson bug that was fixed in 0.52: https://mesonbuild.com/Release-notes-for-0-52-0.html#improved-support-for-static-libraries Bump the requirement and remove the extract_all_objects workaround. This gets rid of a meson warning: WARNING: extract_all_objects called without setting recursive keyword argument. Meson currently defaults to non-recursive to maintain backward compatibility but the default will be changed in the future.	Nirbheek Chauhan	2 years ago
2161	demos: port to Gtk3 demos: port to Gtk3 GTK2 has reached end of life, and GTK3 has been available for a almost a decade. Signed-off-by: Manuel Stoeckl <code@mstoeckl.com> Reviewed-by: Simon Ser <contact@emersion.fr>	Manuel Stoeckl	2 years ago
2160	added aarch64 bilinear implementations (ver.4.1) added aarch64 bilinear implementations (ver.4.1) Since aarch64 has different neon syntax from aarch32 and has no support for (older) arm-simd, there are no SIMD accelerations for pixman on aarch64. We need new implementations. This patch also contains Ben Avions's series of patches for aarch32 and now the benchmark results are fine to aarch64. Please find the result at the below ticket. Added: https://bugs.freedesktop.org/show_bug.cgi?id=94758 Signed-off-by: Mizuki Asakura <ed6e117f@gmail.com>	Mizuki Asakura	3 years ago
2159	Constify region APIs Constify region APIs This allows callers to pass around const Pixman region in their APIs, improving type safety and documentation. Signed-off-by: Simon Ser <contact@emersion.fr>	Simon Ser	3 years ago
2158	tests: Fix undefined symbol build error on macOS tests: Fix undefined symbol build error on macOS prng_state and prng_state_data are getting classified as a "Common symbol" by the compiler due to the convoluted way in which it is `#include`-ed in various test sources, and that's not read as a valid symbol by the linker later. Initializing the symbol clarifies it to the compiler that this specific declaration is the canonical location for this variable, and that it's not a "Common symbol". Fixes https://gitlab.freedesktop.org/pixman/pixman/-/issues/42	Nirbheek Chauhan	3 years ago
2157	Fix -Wincompatible-function-pointer-types warning Fix -Wincompatible-function-pointer-types warning Adding const to the return type does nothing and means that the function pointer types do not match exactly: error: incompatible function pointer types passing 'const float (int, int)' to parameter of type 'dither_factor_t' (aka 'float (*)(int, int)')	Alex Richardson	3 years ago
2156	Fix masked pixel fetching with wide format Fix masked pixel fetching with wide format In __bits_image_fetch_affine_no_alpha and __bits_image_fetch_general, when `wide` is true, the mask is actually an array of argb_t instead of the array of uint32_t it was cast to, and the access to `mask[i]` does not correctly detect when the pixel is nontrivial. The code now uses a check appropriate for argb_t when `wide` is true. One caveat: this new check only skips entries when the mask pixel data is binary all zero; this misses cases like `-0.f` which would be caught by the FLOAT_IS_ZERO macro. As the mask check only appears to be a performance optimization to avoid loading inconsequential pixels, it erring on the side of loading more pixels is safe. Signed-off-by: Manuel Stoeckl <code@mstoeckl.com>	Manuel Stoeckl	3 years ago
2155	Fix signed-unsigned semantics in reduce_32 Fix signed-unsigned semantics in reduce_32	Heiko Lewin	3 years ago
2154	Fix AltiVec detection on FreeBSD. Fix AltiVec detection on FreeBSD.	pkubaj	3 years ago
2153	Avoid out-of-bounds read when accessing individual bytes fro... Avoid out-of-bounds read when accessing individual bytes from mask. The important changes here are a handful of places where we replace memcpy(&m, mask++, sizeof(uint32_t)); or similar code with uint8_t m = mask++; because we're only supposed to be reading a single byte from mask, and accessing a 32-bit value may read out of bounds (besides that it reads values we don't actually want; whether this matters would depend exactly how the value in m is subsequently used). I've also changed a bunch of other places to use this same pattern (a local 8-bit variable) when reading individual bytes from the mask; the code was inconsistent about this, sometimes casting the byte to a uint32_t instead. This makes no actual difference, it just seemed better to use a consistent pattern throughout the file.	Jonathan Kew	3 years ago
2152	meson: update option descriptions. meson: update option descriptions. - gtk is only used in demos - libpng is only used in tests - openmp is only used in tests (in the standard build)	Érico Rolim	3 years ago
2151	meson: remove pixman dependency meson: remove pixman dependency AFAICT from the git history, what happened is that the gtk demos rely on gtk being built with pixman support. pkg-config isn't really expressive enough to have that information, so the solution that was come up with was to search for pixman as well as gtk+ and hope that pixman being installed was. This isn't actually used anywhere in the meson build anyway, and it's causing problems for projects that want to use pixman as a supproject (there's a port of cairo underway that's hitting this), because it confuses meson.	Dylan Baker	4 years ago
2150	meson: add option to skip building of tests and demos meson: add option to skip building of tests and demos Reviewed-by: Dylan Baker <dylan@pnwbakers.com>	Tim-Philipp Müller	4 years ago
2149	meson: add cpu-features-path option for Android meson: add cpu-features-path option for Android Add option to include cpu-features.[ch] from a given path into the build for platforms that don't provide this out of the box. This is needed on Android. Reviewed-by: Dylan Baker <dylan@pnwbakers.com>	Tim-Philipp Müller	4 years ago
2148	Update README a little Update README a little - bugzilla -> gitlab - convert links to https - suggest issues and patches be filed via gitlab	Tim-Philipp Müller	4 years ago
2147	Add -ftrapping-math to default cflags Add -ftrapping-math to default cflags This should resolve https://gitlab.freedesktop.org/pixman/pixman/-/issues/22 and make the tests pass with clang. -ftrapping-math is already the default[1] for gcc, so this should not change behavior when compiling with gcc. However, clang defaults[2] to -fno-trapping-math, so -ftrapping-math is needed to avoid floating-point expceptions when running the combiner and stress tests. The root causes of this issue is that that pixman-combine-float.c guards floating-point division operations with a FLOAT_IS_ZERO check e.g. if (FLOAT_IS_ZERO (sa)) f = 1.0f; else f = CLAMP (da / sa); With -fno-trapping-math, the compiler assumes that division will never trap, so it may re-order the division and the guard and execute the division first. In most cases, this would not be an issue, because floating-point exceptions are ignored. However, these tests call enable_divbyzero_exceptions() which causes the SIGFPE signal to be sent to the program when a divide by zero exception is raised. [1] https://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html [2] https://clang.llvm.org/docs/UsersManual.html#controlling-floating-point-behavior	Tom Stellard	4 years ago
2146	Prevent empty top-level declaration Prevent empty top-level declaration The expansion of PIXMAN_DEFINE_THREAD_LOCAL(...) may end in a function definition, so the following semicolon is considered an empty top-level declaration, which is not allowed in ISO C. Reviewed-by: Matt Turner <mattst88@gmail.com>	Michael Forney	4 years ago
2145	Post-release version bump to 0.40.1 Post-release version bump to 0.40.1 Signed-off-by: Matt Turner <mattst88@gmail.com>	Matt Turner	4 years ago

Older »