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

更改了项目结构,更改了大部分文件,修复了一些问题 #2

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

firemeatman
Copy link

  1. 使用的时候顺便将项目改成了CMake模块化的结构;
  2. 修改了一些细节,包括调整protocol.h文件中json结构的定义顺序、dump()方法获取json字符串内容时非UTF8字符报错的问题;
  3. 客户端加了一个获取词语列表的接口;

firemeatman and others added 10 commits July 15, 2024 00:58
2. 添加获取词语列表的api和json定义;
3. 对测试和cmake进行调整;
-更改MapHandel类,使之接口线程安全,通过访问者模式访问;
-修复管道没有关闭导致线程不能正常退出的Bug;
- 修改protocol文件中的Json序列化,将声明和定义分离,解决第三方引用时可能造成的编译问题,在Qt上的编译已通过;
- 更改一些类的位置;
@firemeatman firemeatman changed the title 更改了项目结构,调整了一些细节问题 更改了项目结构,更改了大部分文件,修复了一些问题 Jul 25, 2024
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