https://bugs.kde.org/show_bug.cgi?id=518931

Nicholas LaPointe <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap
                   |hics/krita/-/commit/0b43971 |hics/krita/-/commit/e685add
                   |41c3eedf2699a24e256d962cc6d |c17238cf64e5d222692e256890d
                   |060a70                      |7b1de7

--- Comment #5 from Nicholas LaPointe <[email protected]> ---
Git commit e685addc17238cf64e5d222692e256890d7b1de7 by Nicholas LaPointe.
Committed on 01/11/2026 at 18:56.
Pushed by nicholasl into branch 'krita/6.0'.

More closely follow GIMP's standard when loading GIMP palettes

Though GIMP's standard[1] mentions that the "the [palette] format was
never formally versioned until 2023", there have thus far been two
versions, with version 2 first appearing in February 2001[2][3].

Krita only supported loading version 2 of the palette format and had
some strange validity checks that didn't quite make sense.
This commit brings support for loading version 1 palettes and changes
the behavior to be closer to GIMP's own palette loading code.[4]

This commit also fixes a crash that could be caused by loading a
palette with a "Name" or "Columns" line that had a missing delimiter.

[1] https://developer.gimp.org/core/standards/gpl/
[2] Introduction of "Name: " attribute
(https://gitlab.gnome.org/GNOME/gimp/-/commit/68216c6ff9b908f0b62bd8b0553cef39a77d55d2)
[3] Introduction of "Columns: " attribute
(https://gitlab.gnome.org/GNOME/gimp/-/commit/267894970de15f8359e796c5147747b545531a0f)
[4]
https://gitlab.gnome.org/GNOME/gimp/-/blob/d8d30252dd7b3dace58233a1b8cfc87cabcd1a63/app/core/gimppalette-load.c#L102
(cherry picked from commit 88f48399e51c72c28e08bb88cc50f580678ee8a0)

M  +28   -35   libs/pigment/resources/KoColorSet.cpp

https://invent.kde.org/graphics/krita/-/commit/e685addc17238cf64e5d222692e256890d7b1de7

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to