Abnormal-AT-segment-identification fixing#17
Merged
TaranDahl merged 3 commits intoJun 12, 2026
Merged
Conversation
Collaborator
|
_find_full_name_mentions 对每个 text segment 都会遍历全部缓存成员做 re.finditer。如果群组成员达到上千人,每条含 @ 的文本都会产生 O(n) 的正则扫描。对于高频的 QQ/Discord 消息转发场景可能有性能影响。 |
WhiteFeather127
added a commit
to WhiteFeather127/QQ-DC-Bridge
that referenced
this pull request
Jun 12, 2026
…apsulation, test coverage, parsing, retry limit
Contributor
Author
|
已修复,详见 commit e5386ec。 优化了
Fixed in e5386ec. Optimized
|
TaranDahl
pushed a commit
that referenced
this pull request
Jun 12, 2026
* binding-and-user-name-mapping * fix PR #17 review issues: race condition, _cache encapsulation, test coverage, parsing, retry limit * Disable QQ→Discord bind-by-nickname feature Comment out the _parse_bind_target path that allowed QQ users to bind to a Discord user by display name. The original code is preserved as comments for reference. PR review (TaranDahl): this feature is unnecessary; removing it also eliminates the related name-matching issue (#4 in PR #16 review). --------- Co-authored-by: WhiteFeather127 <whitefeather127@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix #7