Skip to content

kopt: fix Invert Document regression with Reflow/Auto Straighten#15123

Merged
Frenzie merged 1 commit into
koreader:masterfrom
Euphoriyy:invert-document
Mar 17, 2026
Merged

kopt: fix Invert Document regression with Reflow/Auto Straighten#15123
Frenzie merged 1 commit into
koreader:masterfrom
Euphoriyy:invert-document

Conversation

@Euphoriyy

@Euphoriyy Euphoriyy commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Due to changes to renderOptimizedPage(...) made by #15067, which was merged after #14954 was made, a regression was introduced that causes instability and even crashes on mobile devices and e-readers when Invert Document is enabled along with Reflow or Auto Straighten.

This PR fixes the regression and simplifies the drawContextPage(...) method.


This change is Reviewable

@Frenzie Frenzie added this to the 2026.02 milestone Mar 17, 2026
@Frenzie Frenzie added the bug label Mar 17, 2026
@Frenzie Frenzie merged commit 049e63c into koreader:master Mar 17, 2026
4 of 5 checks passed
jqs7 pushed a commit to jqs7/koreader that referenced this pull request Mar 19, 2026
…eader#15123)

Due to changes to renderOptimizedPage(...) made by koreader#15067, which was merged after koreader#14954 was made, a regression was introduced that causes instability and even crashes on mobile devices and e-readers when Invert Document is enabled along with Reflow or Auto Straighten.

This PR fixes the regression and simplifies the drawContextPage(...) method.
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
…eader#15123)

Due to changes to renderOptimizedPage(...) made by koreader#15067, which was merged after koreader#14954 was made, a regression was introduced that causes instability and even crashes on mobile devices and e-readers when Invert Document is enabled along with Reflow or Auto Straighten.

This PR fixes the regression and simplifies the drawContextPage(...) method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants