- User Since
- Jul 27 2015, 2:36 PM (164 w, 6 d)
Aug 24 2018
OK, let's do it :)
Aug 20 2018
Having a clean solution to this would indeed be great.
Yes please :)
Aug 17 2018
This looks generally nice, I like it. Since we're short on people maintaining this stuff, I'd like more of the code to be shared though.
Looks nice, but I think it would be nice to have captions for the icons. The same is true for the normal OSD, there it shows some caption on hover at least.
Aug 13 2018
Aug 12 2018
Aug 2 2018
Jul 31 2018
Remove more cruft
seems like this is not wanted
Jul 30 2018
Jul 28 2018
The alternative would be to deprecate this and add a new function. I think this is nonetheless sensible.
Jul 27 2018
I think the qml code should not do logic fiddling with the EDID :) but needs investigation.
Messed up merge :(
Add note about the edid pointer becoming invalid
initializer order fixed
Move two more members to the right spot
arc messup, it squashed another patch into this one
arc mangled two patches again :(
Updated commit message and rebased.
Sort edids in test rows. I still think this is completely pointless and a waste of time.
Good catch, I'll have to look into it, right now I get an empty string for both.
Removed xfail, it was fixed by the string parsing fix
Moved Q_REQUIRED_RESULT to the front
It's been there for ages, it's mentioned in the changelog at least... but actually it seems more common to put the attribute before the function, I'll update with that.
I'll check, if it's not public API, I'll make it. Thanks for the pointer.
Yes, my laptop screen uses all 13 bytes, that's how I noticed (the model had one character cut off and I happened to debug and see that it was there, so I did the math :)).
Added one more edid
Removed some extra parenthesis
D14418 is nicer
Remove extra newline
Simplify gamma parsing code
Jul 26 2018
Add more explanations
Only touch saveTimer
Initialize m_changeCompressor to nullptr
Fix the layout
The alignment previous to this commit is also not given, it just happens to look a hint better.
Of course it's questionable if this is worth it, it's unlikely that we'll have so many instances of it that it'll ever make a difference.
Looks good, please finish up cleanup/indentation and I'll accept.
Updated message hopefully
And I see that arc just threw away my updated commit message :(, that's just weird.