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

Fix windows debug crash #162

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Fix windows debug crash #162

merged 1 commit into from
Nov 25, 2024

Conversation

wangfenjin
Copy link
Owner

close #158

@SageMik @xMuscleCodingx 可以帮忙测试下这个改动

@xMuscleCodingx
Copy link

xMuscleCodingx commented Nov 25, 2024

我尝试了下面两种情况:
1、完整的zdic.txt,m1 pro mac没问题,windows崩溃用不了。
2、删除所有注释的zdic.txt,m1 pro mac没问题,windows崩溃用不了。

总结下来现在的情况:
1、 @SageMikv4 (删除注释的 zdic.txt + unsigned char)的windows是没问题的,但是m1 pro mac会乱码且拼音字母搜不到。v4 (删除注释的 zdic.txt + unsigned char)这个里面的zdic.txt还是有注释的
2、不加unsigned,那m1 pro mac就没问题,但是windows崩溃用不了

@wangfenjin
Copy link
Owner Author

总之问题就是用这个分支,zdic.txt 在 windows 下用不了呗;崩溃有什么日志吗

@wangfenjin
Copy link
Owner Author

unsigned char 那个别再提了,求您了,已经确认了那样搞会有问题

@SageMik
Copy link
Contributor

SageMik commented Nov 25, 2024

@xMuscleCodingx 大概看错分支了吧,master 是有注释的,但 v4 是确实是没有注释的。

确定 删除了注释的 zdic + 这个PR的改动 还是会在 Windows 上崩溃吗?我看目前这个改动,按理说删了注释应该就能正常使用了。如果还是崩溃,感觉挺奇怪的

@wangfenjin
Copy link
Owner Author

@xMuscleCodingx 大概看错分支了吧,master 是有注释的,但 v4 是确实是没有注释的。

确定 删除了注释的 zdic + 这个PR的改动 还是会在 Windows 上崩溃吗?我看目前这个改动,按理说删了注释应该就能正常使用了。如果还是崩溃,感觉挺奇怪的

是,我也检查了代码,也看了 zdic 文件,没理由崩溃

@xMuscleCodingx
Copy link

总之问题就是用这个分支,zdic.txt 在 windows 下用不了呗;崩溃有什么日志吗

在electron中用better-sqlite3插入数据时就只出现这一行提示,然后就退出了:
[14232:1125/133349.037:ERROR:crashpad_client_win.cc(868)] not connected

我又单独下载了最新版的sqlite3的exe文件,单独在里面执行测试的sql语句,同样是执行插入语句的时候就退出了。

我测试用的这个改动+zdic删除所有注释版本里的windows x64。

@wangfenjin
Copy link
Owner Author

那我建议你还是用自带的 pinyin.txt 吧,是有什么特别的理由非要用 zdic 吗

@xMuscleCodingx
Copy link

好的,辛苦二位了 @wangfenjin @SageMik ,感谢!

想换的原因就是自带的pinyin.txt太全了,例如之前提到的"有"字,在pinyin.txt中有三个读音"yǒu,yòu,wěi",这就导致输入"w"会匹配到"有"字,就感觉不太正常。

@wangfenjin wangfenjin merged commit 632fd7b into master Nov 25, 2024
8 checks passed
@wangfenjin wangfenjin deleted the fix-debug-crash branch November 25, 2024 07:58
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.

在 build_type 为 debug 的情况下,Windows 下编译出来的 dll 遇到中文会崩溃
3 participants