]> git.mdlowis.com Git - proto/labwc.git/commit
rcxml: discourage empty strings in rc.xml configuration
authortokyo4j <hrak1529@gmail.com>
Wed, 3 Dec 2025 09:26:44 +0000 (18:26 +0900)
committerHiroaki Yamamoto <hrak1529@gmail.com>
Wed, 3 Dec 2025 20:02:26 +0000 (05:02 +0900)
commit06505d24c8d3a5e5fa1d0212789aa104ff97b8ed
tree310348689fca4f77d699f47b9dff31a64aac8bc8
parenta5db8e477aec3cb96044805d25454bd97af94342
rcxml: discourage empty strings in rc.xml configuration

Background:
I rewrote the config parser in 9462457..2f414a4, but it broke certain
configurations by changing how empty strings are handled: they were mostly
just ignored before my parser rewrite, but after that, they are
interpreted as just empty strings (output="" is considered as 'output named ""').

Though that was unintentional, I believe ignoring empty strings was not a
good idea in the first place, as we already allow empty strings for
certain configurations (e.g. `<desktop prefix="">`), which makes the
parser's behavior inconsistent.

Change:
So let's clarify that we intend to read empty strings as empty strings.
As a preparation, this commit adds warnings for empty strings we are
currently ignoring, so that users can be informed that we intend to just
read empty strings (e.g. `<theme name="">`) as empty strings in the future.
I removed existing empty strings in `rc.xml.all` to avoid warnings when
reading it.
docs/rc.xml.all
src/config/rcxml.c