kernel: expand CP850 macro #487

Merged
Burer merged 2 commits from ace-dent/kolibrios-gitea:PR-Encoding-1 into main 2026-06-13 04:12:16 +00:00
Member
  • Add more detailed (and precise) comments.
  • Expand cp850 macro to encode more code points; useful for German characters ä and ü.
  • Note: all of the current CP850 encodings map directly to CP437.
- Add more detailed (and precise) comments. - Expand `cp850` macro to encode more code points; useful for German characters `ä` and `ü`. - Note: all of the current CP850 encodings map directly to CP437.
ace-dent added 1 commit 2026-06-09 16:24:26 +00:00
kernel: expand CP850 macro
Build system / Check kernel codestyle (pull_request) Successful in 50s
Build system / Build (pull_request) Successful in 17m16s
46839bf81a
- Add more detailed (and precise) comments.
- Expand `cp850` macro to encode more code points; useful for German characters `ä` and `ü`.
- Note: all of the current CP850 encodings map directly to CP437.
ace-dent requested review from Burer 2026-06-09 16:24:26 +00:00
ace-dent requested review from Doczom 2026-06-09 16:24:26 +00:00
ace-dent requested review from dunkaist 2026-06-09 16:24:27 +00:00
ace-dent requested review from Sweetbread 2026-06-09 16:24:27 +00:00
Burer added a new dependency 2026-06-09 16:50:03 +00:00
Author
Member

This PR is necessary to fix some typos on the German de_DE boot screen, which will be addressed by #485 .

This PR is necessary to fix some typos on the German `de_DE` boot screen, which will be addressed by #485 .
Burer added 1 commit 2026-06-10 12:59:33 +00:00
kernel/encoding: clarify CP437/850 comment
Build system / Check kernel codestyle (pull_request) Successful in 29s
Build system / Build (pull_request) Successful in 9m53s
ea4bea4178
Burer approved these changes 2026-06-10 13:03:53 +00:00
Leency approved these changes 2026-06-11 20:15:38 +00:00
Author
Member

@Burer - I don't want to stall merging, but the comment you changed was a useful note to future developers (and me!). Were you concerned it wasn't accurate?

@Burer - I don't want to stall merging, but the comment you changed was a useful note to future developers (and me!). Were you concerned it wasn't accurate?
Owner

@ace-dent

Yes, many codepoints in initial range are actually different for CP437 and C850 - mosty symbols for box drawing, that are not present in CP850.

image.png

@ace-dent Yes, many codepoints in initial range are actually different for CP437 and C850 - mosty symbols for box drawing, that are not present in CP850. ![image.png](/attachments/009eb804-4423-44b0-aeb7-51a9396cde0a)
Author
Member

@Burer - thanks for the info. Looks good to merge! 🙌

@Burer - thanks for the info. Looks good to merge! 🙌
Burer merged commit 208e157e04 into main 2026-06-13 04:12:16 +00:00
Burer deleted branch PR-Encoding-1 2026-06-13 04:12:16 +00:00
ace-dent removed the
PR
Review required
label 2026-06-18 12:01:59 +00:00
ace-dent removed review request for Doczom 2026-06-18 12:02:15 +00:00
ace-dent removed review request for dunkaist 2026-06-18 12:02:17 +00:00
ace-dent removed review request for Sweetbread 2026-06-18 12:02:19 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Blocks
#485 kernel: fix minor typos
KolibriOS/kolibrios
Reference: KolibriOS/kolibrios#487