| 5 Oct 2025 |
K900 | Yeah that sounds very wrong tbh | 16:37:59 |
K900 | Have you tried profiling it maybe | 16:38:10 |
K900 | 125 here | 16:39:32 |
K900 | Inside Yakuake | 16:39:32 |
K900 | Which is itself a wrapper | 16:39:33 |
Marie | no, what would be the tool i want to use to do that? | 16:42:04 |
K900 | perf | 16:43:18 |
Marie | this is from profiling plasmashell and repeatedly clicking on the audio menu | 17:21:35 |
Marie | Download plasma-thing.txt | 17:21:35 |
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 |