Bug 155544 - Tools - Options - Application Color - change "Automatic:" value: NIL color config warnings in console with debug LO
Summary: Tools - Options - Application Color - change "Automatic:" value: NIL color co...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
7.6.0.0 alpha1+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Options-Dialog-Colours
  Show dependency treegraph
 
Reported: 2023-05-29 11:52 UTC by BogdanB
Modified: 2023-08-03 18:46 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description BogdanB 2023-05-29 11:52:07 UTC
Description:
In debug LibreOffice:
Go to Tools - Options - Application Color - and from Automatic choose: Dark and Apply. 

This warning is listed several times:
warn:legacy.osl:349899:349899:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!

Should not!

Steps to Reproduce:
See description

Actual Results:
Warnings

Expected Results:
No warnings


Reproducible: Always


User Profile Reset: Yes

Additional Info:
Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 2e57755f72907e4bb604a8ba32edbd6c253ee57c
CPU threads: 4; OS: Linux 6.2; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded
Comment 1 Julien Nabet 2023-05-29 16:44:24 UTC
On pc Debian x86-64 with master sources updated today, I could reproduce this.
The colors correspond to the 11 present in ./officecfg/registry/data/org/openoffice/Office/ExtendedColorScheme.xcu for reportbuilder.

Just wonder if we should include these in:
- cui/source/options/optcolor.cxx
- svtools/source/config/colorcfg.cxx
...
with #ifdef ENABLE_REPORTBUILDER/#endif blocks.

Remark: it's not specific to Dark, it's just when you change theme.
For example, if I clean the LO profile, LO uses by default "Light". If I change it to "System Theme", I also got these console logs.
Also, idem if I put back "Light".
Comment 2 Julien Nabet 2023-05-29 16:50:21 UTC
BTW, I don't understand what means "Automatic:" here.
If it just corresponds to the scheme value, it's a bit weird. I would have put "Option".
Scheme "Automatic" with Option "Light", "Dark" or "System Theme".
Even "Automatic" value for scheme is not clear for me, moreover since I got only 1 value for "Scheme", it doesn't help.
Comment 3 Heiko Tietze 2023-06-02 08:47:16 UTC
The whole ExtendedColorScheme.xcu is weird. Do we need it at all?

(In reply to Julien Nabet from comment #2)
> BTW, I don't understand what means "Automatic:" here.

We have hard-coded default values for Light and Dark. "Automatic" means we take those and not the user-defined. But you are right, there is plenty of room for improvements.
Comment 4 Julien Nabet 2023-06-04 18:57:32 UTC
(In reply to Heiko Tietze from comment #3)
> The whole ExtendedColorScheme.xcu is weird. Do we need it at all?
No idea


> 
> (In reply to Julien Nabet from comment #2)
> > BTW, I don't understand what means "Automatic:" here.
> 
> We have hard-coded default values for Light and Dark. "Automatic" means we
> take those and not the user-defined. But you are right, there is plenty of
> room for improvements.
Ok so would rather be something like "By default"
Comment 5 Heiko Tietze 2023-06-05 06:47:09 UTC
Exchanging "Automatic" into "Default" is not much of an improvement and not solving Bogdan's finding anyway.

Adding a compiler switch solves the issue in most but not all cases, adding the extended colors to the list would be okay as well as dropping it completely.
Comment 6 BogdanB 2023-08-03 18:46:46 UTC
Retested with
Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 0e17d8ddccdee7d4f46b256153853e1cab87c99e
CPU threads: 16; OS: Linux 5.19; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded

warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!
warn:legacy.osl:1071126:1071126:svtools/source/config/extcolorcfg.cxx:353: Color config entry has NIL as color value set!