Skip to content

Lua blitter: Fix the mul method for grayscale Color data types#1922

Merged
NiLuJe merged 1 commit into
koreader:masterfrom
NiLuJe:master
Sep 7, 2024
Merged

Lua blitter: Fix the mul method for grayscale Color data types#1922
NiLuJe merged 1 commit into
koreader:masterfrom
NiLuJe:master

Conversation

@NiLuJe

@NiLuJe NiLuJe commented Sep 7, 2024

Copy link
Copy Markdown
Member

We want to access the actual luminance value instead of the struct pointer ;o).

Fix koreader/koreader#12473


This change is Reviewable

We want to access the actual luminance value instead of the struct
pointer ;o).

Fix koreader/koreader#12473
@NiLuJe

NiLuJe commented Sep 7, 2024

Copy link
Copy Markdown
Member Author

At a quick re-re-review, I don't appear to have bungled this up anywhere else ;).

Not quite sure how I missed it in the first place, as I did test the Lua blitter back in January, on both color & grayscale BBs.

┌─(niluje@ahsoka:pts/14)────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────(/var/tmp/niluje/koreader)─┐
└─(0.79:19%:16:44:%)── h 1 | rg EMULATE_BB_TYPE                                                                                                                                                                                                                                                                                                        ──(Sat, Sep 07)─┘
 2378  23.1.2024 13:17  0:00  env EMULATE_BW_SCREEN=1 EMULATE_BB_TYPE="BB8" ./kodev run
10086  7.9.2024 16:40  0:30  env EMULATE_BW_SCREEN=1 EMULATE_BB_TYPE="BB8" ./kodev run

Must have implemented the MUL stuff later or something... -_-".

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.

Highlighting issues

2 participants