Skip to content

Fix the rank of 子 as ㄗㄜˇ - #765

Merged
zonble merged 1 commit into
openvanilla:masterfrom
lukhnos:fix-ze3-rank
Jan 1, 2026
Merged

Fix the rank of 子 as ㄗㄜˇ#765
zonble merged 1 commit into
openvanilla:masterfrom
lukhnos:fix-ze3-rank

Conversation

@lukhnos

@lukhnos lukhnos commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

The reading was introduced in #644, and this ensures that 怎 always comes first.

The reading was introduced in openvanilla#644,
and this ensures that 怎 always comes first.
Copilot AI review requested due to automatic review settings December 29, 2025 21:24

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adjusts the pronunciation ranking for the character '子' by adding it to heterophony1.list with its common pronunciation ㄗˇ and to heterophony3.list with the rarer pronunciation ㄗㄜˇ. This change correctly uses the heterophony ranking system to influence candidate frequency in the IME. My main feedback is regarding the structure of the data files. Both modified heterophony*.list files are not sorted, while the project's Makefile includes a sort target for this purpose. Keeping these files sorted improves maintainability and helps prevent duplicate entries. I've added comments to the respective files with this suggestion.

Comment thread Source/Data/heterophony1.list
Comment thread Source/Data/heterophony3.list

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes the ranking of the character 子 when typed with the Bopomofo reading ㄗㄜˇ by adding it to the heterophony3.list, ensuring that 怎 (a more common character with the same reading) appears first in the candidate list.

  • Adds 子 ㄗˇ to heterophony1.list as the default/common reading
  • Adds 子 ㄗㄜˇ to heterophony3.list as a less common reading

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Source/Data/heterophony1.list Adds the default reading 子 ㄗˇ, maintaining C locale sort order
Source/Data/heterophony3.list Adds the rare reading 子 ㄗㄜˇ, maintaining C locale sort order to ensure 怎 has higher priority

@zonble
zonble merged commit 35faa87 into openvanilla:master Jan 1, 2026
14 checks passed
@lukhnos
lukhnos deleted the fix-ze3-rank branch January 11, 2026 00:52
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.

3 participants