Skip to content

Use Set for better performance#47

Draft
graphemecluster wants to merge 1 commit into
mainfrom
set
Draft

Use Set for better performance#47
graphemecluster wants to merge 1 commit into
mainfrom
set

Conversation

@graphemecluster

Copy link
Copy Markdown
Member

Perhaps?

@syimyuzya

Copy link
Copy Markdown
Member

I don't think a Set with just tens of elements at most would have a performance improvement that beats simplicity 🤔

However we probably do need reverse lookups for 呼韻搭配, 等母搭配 etc, so we can eliminate some for-loops in 音韻地位.驗證.

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