| 5 Oct 2025 |
 K900 ⚡️ | Can you post the raw recording | 17:22:58 |
| K900 ⚡️ | Or upload it to https://profiler.firefox.com/ | 17:23:38 |
| K900 ⚡️ | (it can actually do arbitrary perf profiles, not just Firefox) | 17:23:51 |
 Marie | It throws an error when I try to upload it Error: Unserializing the profile failed: Error: The profile array buffer could not be parsed as a UTF-8 string. | 17:27:07 |
| Marie | oh i should read the docs | 17:27:53 |
| Marie | https://share.firefox.dev/46Js6H2 | 17:31:13 |
| K900 ⚡️ | Hmm | 17:33:43 |
| K900 ⚡️ | I wonder if we can somehow save all the syncColors calls | 17:33:50 |
| K900 ⚡️ | Actually HUH | 17:34:57 |
| K900 ⚡️ | https://github.com/KDE/qqc2-desktop-style/commit/89760fddd4cbfaa942847cc1895a63bb789bfab8 | 17:34:58 |
| K900 ⚡️ | It should be cached | 17:35:01 |
| K900 ⚡️ | I wonder why we're not hitting that | 17:35:06 |
| K900 ⚡️ | Or is it because it's passing in different groups every time | 17:39:14 |
| K900 ⚡️ | And hitting a full filesystem lookup | 17:39:20 |
| K900 ⚡️ | The good news is that it should be easy to fix | 17:42:01 |
| K900 ⚡️ | The bad news is that I'm definitely not doing it in, like, at least the next two cycles | 17:42:14 |
| K900 ⚡️ | AT LEAST | 17:42:29 |
| K900 ⚡️ | I guess the giga brain solution would be to add a cache to QStandardPaths | 17:44:00 |
| K900 ⚡️ | But that is big enough that you'd have to think about like, eviction policies | 17:44:14 |
| Marie | yeah don't worry, it's not like this has any urgency | 17:45:31 |
| K900 ⚡️ | And strictly speaking KColorScheme can still cache better | 17:45:37 |
| Marie | and with all of the fire going on... | 17:45:39 |
| K900 ⚡️ | Because QStandardPaths will only cache file lookups | 17:45:43 |
| K900 ⚡️ | * Because QStandardPaths will only cache file path lookups | 17:45:48 |
| K900 ⚡️ | And KColorScheme can cache the entire color scheme | 17:45:54 |
| K900 ⚡️ | And given the file size of an average color scheme you can probably read all of it in one syscall anyway | 17:46:31 |
| Marie | is that the right syncColors?
https://github.com/KDE/libplasma/blob/e39d72b9ab8c07b6bd76bcd1e89e1a8efef74df2/src/declarativeimports/kirigamiplasmastyle/plasmatheme.cpp#L110
is it not this one? or does one thing call the other? | 18:04:51 |
| K900 ⚡️ | Actually it might be | 18:06:17 |
| K900 ⚡️ | But it seems to have the same problem | 18:06:21 |
| 7 Oct 2025 |
| K900 ⚡️ | @outfoxxed holy shit 6.10 actually stabilizes the on-disk cache format | 12:44:13 |
