dallasgroot/Gimp
dallasgroot/Gimp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Issue #16169: do not serialize empty string value.

Browse source 
Reverting a part of commit fa097df9 which should not have been merged,
and in fact which was not useful anymore for fixing the original issue.

This fixes part of #16169, which is that because of this change, we were
creating broken XCF containing both a text and a markup variant for
text-only text layers. Typically it was storing:

> (text "I love GIMP")
> (markup "")

Now we don't create such broken XCF anymore, but I'll want to strengthen
reading code too, so that we can salvage such XCF files, already
existing in the wild.
This commit is contained in:
Jehan 2026-04-08 15:46:23 +02:00
parent edc7a14127
commit aa7c2a55d6
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
libgimpconfig/gimpconfig-serialize.c
View file

@ -576,10 +576,13 @@ gimp_config_serialize_value (const GValue *value,
{
const gchar *cstr = g_value_get_string (value);
if (! cstr)
return FALSE;
if (escaped)
gimp_config_string_append_escaped (str, cstr ? cstr : "");
gimp_config_string_append_escaped (str, cstr);
else
g_string_append (str, cstr ? cstr : "");
g_string_append (str, cstr);
return TRUE;
}