You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
mcdurdin
changed the title
bug(common): fix handling of CODE_CONTEXTEX offsets in Developer and Core
bug(common): fix handling of CODE_CONTEXTEX and CODE_INDEX offsets in Developer, Core, and Web
Feb 21, 2025
Add validity checks to kmcmplib to verify that offsets provided to
`context(n)` are in range for the context, and do not point to
non-character elements (`nul` and `if`) in the context string.
Fixes: #13301Fixes: #13302
Relates-to: #13299
Relates-to: #13276
context(n)
counts thenul
statement as a character in Web & android #13276 issue all function correctly with updated Keyman Corecontext(n)
in the context part of the rule as a follow-on for this.notany()
to the unit test very late. This bumped the minver of the keyboard to 14.0, which caused the compiler to use v10+ rules, so the rules withnul
no longer worked (bug(developer): kmc-kmn kmw compiler generates an incorrect offset fornul ... > context(n)
when in v10+ mode #13306). However I didn't manually retest the keyboard after addingnotany()
so I didn't pick this up.The text was updated successfully, but these errors were encountered: