-Wsign-compare babl-memory

-Wold-style-declaration

-Wsign-compare conversions.c

Fix warning: -Wsign-compare babl-fish-path

babl-lut-verify: warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare]

babl_fish_path: comparison of integer expressions of different signedness

Fix warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare]

bin: profile-aware babl format defaults.

Instead of always defaulting to "R'G'B' float", if a input/output
profile was set, check if it's CMYK, grayscale or RGB.

The defaults are:
* For a CMYK profile: "CMYK float"
* For a grayscale profile: "Y' float"
* For anything else (or no profile): "R'G'B' float"

babl: fix typos in error strings.

bin: add -r/--intent for setting the rendering intent on the CLI tool.

bin: show source format in verbose/default output

bin: add a --brief/-b option.

This will output the result as int or float values (depending on output
format), separated by space.
It would allow to use the output of this script for testing conversion
(with expected output), and can also be used for chaining calls, making
this a re-entrant CLI tool.

bin: improve usage of the CLI tool.

- Add a --help/-h option.
- Detect invalid options (argument starting with '-').
- Support '--' option to indicate end of options. This can be
  particularly useful to set negative component values in floating point
  formats.
- Print usage in various error types (when they are of wrong
  syntax/option types), additionally to printing it when --help is
  called.

bin: convert from Babl formats with space.

bin: new "babl" tool to convert pixel values in command line.

This is a CLI tool to quickly make conversions of raw pixel values from
one known format to another.

This is a first basic version, which supports input and output in float,
u8, u16 and u32 (no "half" type yet in particular).

Also it doesn't support profiles yet, or rendering intents, but soon
will.

Finally it doesn't have a "help" output yet to help usage.

babl/base/babl-rgb-converter.c: Fix universal_nonlinear_rgb_u8_converter_sse2

Fix alignment of vector of floats in babl-rgb-converter.c
Downstream Bug (Gentoo): https://bugs.gentoo.org/857708
Bug: https://gitlab.gnome.org/GNOME/babl/-/issues/76

Thanks-to: Alexander Bezrukov <phmagic@mail.ru>
Signed-off-by: Alexander Bezrukov <phmagic@mail.ru>
Signed-off-by: Sergey Torokhov <torokhov-s-a@yandex.ru>

meson: Various fixes

- Add missing lcms dependencies. That's needed when lcms is a subproject
  otherwise those targets does not find its headers.
- Add lcms2 wrap so meson can build it as subproject in case the
  dependency is not found on system.
- Fix couple meson warnings
- Use meson.override_dependency() so babl can be used as subproject
  without hardcoding "babl_dep" variable name in main project.

meson: fix misspelled kwarg name

set10 doesn't have a `Description` kwarg, it does have a `description`
kwarg though.

This caused the conf variable to not have a description when it should
have one, and newer versions of Meson with better argument validation
error out with:

meson.build:58:5: ERROR: configuration_data.set10 got unknown keyword arguments "Description"

post-release version bump

release 0.1.92

extensions: build arm-neon variants of basic extensions

extensions: simple avoid unneccesary double

CIE: full separation of single and double precision code

babl: prevent loading extension modules with incorrect SIMD

This now happens on top of bailing inside the individual modules -
which in some cases is insufficient. This is likely sufficient
to fix issue #69.

avoid some avoidable implicit promotion to double

ci: disable to save the environment

Arch linux continue to be a bit incompatible with the buildfarm,
CI is already bad for the environment, cpu consuming expected
failures are even worse.

The CI might turn up again backed by debian, but it is better to
turn it off for now.

ci: attempt building with latest archlinux image

tools: mark a function static inline, needed for -O0 builds with clang

meson: post release version bump

release babl 0.1.90

NEWS: update

babl: drop a now deprecated const

babl_format_new: also free doc to plug leak

doc and name are allocated via babl_strdup, hence if we don't use them
we need to free both of them - therefore add a free(doc) to match the
existing free(name).

format_new (called just below) transfers ownership of doc, hence it's
correct not to free doc in the case where we're instantiating a new
format - whereas name is copied within format_new, and therefore needs to be
free'd in either scenario.

Leak seen while running gimp+babl built with ASAN:

  Direct leak of 99 byte(s) in 1 object(s) allocated from:
    #0 0x5e87cd in malloc /home/abuild/rpmbuild/BUILD/llvm-12.0.0.src/build/../projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:145:3
    #1 0x7f038605e2b8 in babl_malloc /home/ahunt/git/babl/_build/../babl/babl-memory.c:115:9
    #2 0x7f038605eec7 in babl_strdup /home/ahunt/git/babl/_build/../babl/babl-memory.c:296:9
    #3 0x7f038605110f in babl_format_new /home/ahunt/git/babl/_build/../babl/babl-format.c:376:17
    #4 0x7f037313a432 in init /home/ahunt/git/babl/_build/../extensions/cairo.c:569:25
    #5 0x7f03860412a1 in babl_extension_load /home/ahunt/git/babl/_build/../babl/babl-extension.c:226:7
    #6 0x7f0386040d1c in babl_extension_load_dir /home/ahunt/git/babl/_build/../babl/babl-extension.c:268:19
    #7 0x7f03860409a8 in babl_extension_load_dir_list /home/ahunt/git/babl/_build/../babl/babl-extension.c:338:17
    #8 0x7f038607d6cd in babl_init /home/ahunt/git/babl/_build/../babl/babl.c:154:7
    #9 0x7f0386bce5ab in gegl_post_parse_hook /home/ahunt/git/gegl/_build/../gegl/gegl-init.c:542:3
    #10 0x7f03844aad8a in g_option_context_parse /home/ahunt/git/glib/_build/../glib/goption.c:2228:12
    #11 0x7f0386bce381 in gegl_init /home/ahunt/git/gegl/_build/../gegl/gegl-init.c:189:8
    #12 0xef1f6b in gimp_init_for_testing /home/ahunt/git/gimp/app/tests.c:69:3
    [...]

SUMMARY: AddressSanitizer: 99 byte(s) leaked in 1 allocation(s).

Add CIE Lab alpha u16 format

Add CIE Lab alpha u8 format

babl: fix mutex lock/unlocks symmetry in babl_fish()

In the case where the fish we are to look up is the memcpy fish we were
not locking the mutex. Whereas the common cleanup after the branches
assumes the lock is held. This is probably the cause of gimp#7632.

tools: include more conversions in babl-lut-verify

tools: use _putenv_s() instead of setenv() on Windows.

setenv() does not exist on Windows API, hence babl tools fail to build.
Fixing the issue by using an alternative Windows API.

tools: rewrite babl-lut-verify to be more generic

tools: include CMYK encodings in babl-benchmark

babl: add 4:8 luts

tools/babl-benchmark: print LUT/REF markers

babl: do not load cache if BABL_INHIBIT_CACHE is set in environment

tools: inhibit cache use in lut-verify and babl-benchmark

sse2-float: correctly handle associted alpha in remainder

babl: fix fix bpp initialization when loaded from cache

babl: clean up compiler warnings

babl: do luts for 2->16 byte fishes

babl: initialize bpp after reading cost in fish cache

babl: make unified alpha conversion branchless

babl: improve LUT debug info

babl: add support for 2,2 LUTs

babl: only show active luts during gc on info_level>=4

new symbol babl_fish_get_process

This permits some APIs to avoid a function call per dispatch,
specifically the samplers in GeglBuffer.

babl: remove optional instrumentation

babl: add LUT debugging configurable through BABL_LUT_INFO environment variable

It is a number ranging from 0 to 5 if with more information provided at
higher levels.

babl: measure performance of LUTs and only replace if faster

With this also enabling replacement of conversions where spaces do not
differ.

babl: LUTs for 16byte destination encodings

babl: do LUTs for 1 and 2 byte encodings

tools: add RGB variants to babl-benchmark

babl: also do auto LUTs for 3-3 bpp and 3-4bpp paths

babl: lut remove unused u16 code

tools: use rand() as fallback of random().

This fixes building babl on Windows as the random() API is not available
there (unlike rand()). Windows docs says it's not cryptographically
secure but for this tool usage, it's probably enough and I see we use
the same function replacement tricks on other tool sources.

Revert "make trampoline for lut processing"

This reverts commit b3e884edf3b5c58fb4c2cede1346bd8a9d9c4a1e.

benchmarking on x86_64 saw no effect, and as a surprise on arm
it pessimises performance.

build: fix append of sse2 cflags

make trampoline for lut processing

build: append list of one element rather than string

tools: add babl-lut-verify

ctx: factor lut processing out of path_process

babl: LUTs fix typo in timelimit check

babl: free LUTs that have been unused for more than 5 minutes

babl: lut, tidy u8 code - disable u16/half variant

babl: avoid using half initialized LUTs when used from threads

babl: add sse2 flags when building RGB converter

babl: LUT cleanups

babl: use LUT for 16bit to 8bit conversions

babl: remove some now dead u8 code

babl: add lut for some u8 conversions

The fish path now marks created fishes that shall be monitored for
usage and have full 24bit LUTs generated on demand when limits are
exceeded. This 3-4x increases the throughput.

tools: rework babl-benchmark to operate on sets resembling workingspace + GEGL + display

babl: add u8 code paths to trc/matrix space conversion

This is relevant for ARM, on modern x86_64 at least it is cheaper to
convert to/from float with external loops.

babl: adjust default path parameters to a path_length of 3

Also adjust so that the max additional bits of depth is 2, this makes us
catch some fast paths that are missed by very promising slow two-step
converters.

babl: remove allocations in u8 variants of RGB space conversions

babl: cpuaccel fix arm neon detect

babl: cpuaccel fix arm neon detect

babl: add more missing SIMD build related files

babl: add missing files babl-trc.[ch]

extensions/cairo: add a correct enough for 8bit cairo32 to cairo24 conversion

extensions: reduce amount of competing conversions

Avoid loading extensions if a variant with better SIMD support
is expected to exist. We know all variants that are expectd to
exist and thus can register only the fastest known variant.

This avoids combinatorial explosion of conversions to test.

tools: babl-verify make verify do space free conversions by default, it is more useful

extensions: cairo remove a broken conversion, fix RGBA u8 to cairo reference

multipass build, with SIMD variants for x86_64 and arm

Some extensions have their own separate build, so does the
RGB color space conversions, trc computations and base
set of conversions.

babl: reduce number of pixels tested for paths to 1024, include some out of bounds pixels in start

base: x86-64-v2 and x86-64-v3 variants

build: enable -ftree-vectorize always, and -mfpu=neon on aarch64

babl: sync cpuaccel with GEGL

gitlab-ci: disable paccache, trying to unbreak build

babl: set ecx to 0 in cpuid

This is neccesary to be able to detect AVX2 reliably.

tests: add a xyz to lab test

This is testing D50 XYZ to D50 CIE Lab the use of D50 for all XYZ and
Lab is currently implicit in all babl code.

Oklab: disable only Oklch not Oklab

Oklab itself is enabled by default now for testing, but does not seem to
produce correct values with the Oklab test vectors, even with manual
attempt at converting the test vector (presumed) CIE XYZ D65 to CIE XYZ
D50 before running the conversion.

The failure to pass symmetry tests as mentioned in issue #63 seems to be
limited to Oklch, Oklab itself is fine.

oklab: add double variant of code for model conversions