Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

opt:refactor wordfinder,use signal/connect to handle single requestFinish event. #2151

Merged
merged 2 commits into from
Feb 20, 2025

Conversation

xiaoyifang
Copy link
Owner

@xiaoyifang xiaoyifang commented Feb 12, 2025

relate #2113

@xiaoyifang xiaoyifang marked this pull request as draft February 13, 2025 05:50
@xiaoyifang xiaoyifang force-pushed the opt/word-finder branch 2 times, most recently from 7ea6fb8 to 30cfd40 Compare February 14, 2025 02:38
@xiaoyifang xiaoyifang force-pushed the staged branch 2 times, most recently from c7d784f to f2fe677 Compare February 17, 2025 06:24
@xiaoyifang xiaoyifang force-pushed the opt/word-finder branch 3 times, most recently from e9fd4fe to fc7f4a4 Compare February 17, 2025 06:29
@xiaoyifang xiaoyifang force-pushed the opt/word-finder branch 3 times, most recently from c411c4d to 736be89 Compare February 20, 2025 06:10
@xiaoyifang xiaoyifang changed the title opt:add mutex to the wordfinder opt:refactor wordfinder,use signal/connect to handle single requestFinish event. Feb 20, 2025
use more specific signal to handle the event
@xiaoyifang xiaoyifang marked this pull request as ready for review February 20, 2025 06:20
Copy link

@xiaoyifang xiaoyifang merged commit 3cea788 into staged Feb 20, 2025
14 checks passed
@xiaoyifang xiaoyifang deleted the opt/word-finder branch February 20, 2025 14:41
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.

1 participant