Skip to content

Expose CorrectHomographNumbers publicly on ILexEntry#375

Open
myieye wants to merge 1 commit into
masterfrom
expose-correct-homograph-numbers
Open

Expose CorrectHomographNumbers publicly on ILexEntry#375
myieye wants to merge 1 commit into
masterfrom
expose-correct-homograph-numbers

Conversation

@myieye
Copy link
Copy Markdown
Contributor

@myieye myieye commented May 13, 2026

Adds ILexEntry.CorrectHomographNumbers() which wraps the existing LexDb.CorrectHomographNumbers algorithm.
FieldWorks Lite writes HomographNumber directly and needs a way to trigger a "cleanup" of a potentially broken homograph sequence.


This change is Reviewable

@github-actions
Copy link
Copy Markdown

LCM Tests

    16 files  ±0      16 suites  ±0   3m 1s ⏱️ +4s
 2 860 tests +2   2 840 ✅ +2   20 💤 ±0  0 ❌ ±0 
11 388 runs  +8  11 220 ✅ +8  168 💤 ±0  0 ❌ ±0 

Results for commit b1e7ff6. ± Comparison against base commit 66da667.

Copy link
Copy Markdown
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

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

From an API standpoint I don't like adding a method on LexEntry which modifies data in other LexEntries. This should be exposed, but I don't think it belongs here.

@jasonleenaylor reviewed all commit messages and made 1 comment.
Reviewable status: 0 of 3 files reviewed, all discussions resolved.

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.

2 participants