Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MXCFB: Unbreak Zelda/Rex #1314

Merged
merged 4 commits into from Feb 23, 2021
Merged

MXCFB: Unbreak Zelda/Rex #1314

merged 4 commits into from Feb 23, 2021

Conversation

NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Feb 23, 2021

* One that actually uses REAGLD only on a select few devices, those where REAGL appears extra finicky (right now, Kindle Zelda & Rex).
* Re-enable FORCE_MONOCHROME on Zelda & Rex. (Anyway, Rex can be lower res, so it somewhat weighs in the balance, too).


This change is Reviewable

Useful on Kindle Zelda/Rex, where REAGL is sometimes too finicky...
Also, actually check the right A2 constant.
Kindles also suffer from similar FORCE_MONOCHROME quirks, so, keep it
disabled there ;).
@NiLuJe NiLuJe marked this pull request as draft February 23, 2021 03:31
Copy link
Member

@yparitcher yparitcher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes Both issues

@NiLuJe
Copy link
Member Author

NiLuJe commented Feb 23, 2021

Thanks, that means I can scrap the whole REAGLD idea :D.

We've also definitely confirmed that I should stop writing these things at 4AM ^^.

@NiLuJe NiLuJe marked this pull request as ready for review February 23, 2021 03:45
@NiLuJe
Copy link
Member Author

NiLuJe commented Feb 23, 2021

And this much simpler version should still fix all the things ;).

@yparitcher
Copy link
Member

You still left some REAGLD stuff (ZELDA_GLD16)

@NiLuJe
Copy link
Member Author

NiLuJe commented Feb 23, 2021

Yeah, can't hurt leaving the actual flag selection logic in, given how prone I am to forgetting that the Zelda/Rex constants are spelled differently ^^.

(This... isn't the first time that same mistake crept in somewhere ^^).

@NiLuJe NiLuJe changed the title MXCFB: Expose a REAGLD refresh method MXCFB: Unbreak Zelda/Rex Feb 23, 2021
@NiLuJe NiLuJe merged commit 86a57a8 into koreader:master Feb 23, 2021
NiLuJe added a commit to NiLuJe/koreader that referenced this pull request Feb 23, 2021
NiLuJe added a commit to koreader/koreader that referenced this pull request Feb 23, 2021
roygbyte pushed a commit to roygbyte/koreader-base that referenced this pull request Mar 3, 2022
* Fix a terrible typo in koreader#1312 that used the wrong A2 constant on Zelda/Rex, causing the Reader to get squished to 2bit by FORCE_MONOCHROME. Ouch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants