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

feat: 增加lua脚本的注释 #88

Open
wants to merge 2 commits into
base: 7.0-cn-annotated
Choose a base branch
from

Conversation

artikell
Copy link

No description provided.

Copy link
Collaborator

@enjoy-binbin enjoy-binbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

貌似很多中文,在我这边看或者编辑,都是乱码,你那边可以检查下吗

src/db.c Outdated Show resolved Hide resolved
@enjoy-binbin
Copy link
Collaborator

@ncghost1 你那边看或者编辑这些中文会乱码吗

@artikell artikell force-pushed the feature/lua-annotated branch 2 times, most recently from 809b0ea to 954263d Compare October 30, 2022 10:07
@ncghost1
Copy link
Member

ncghost1 commented Oct 31, 2022

@ncghost1 你那边看或者编辑这些中文会乱码吗

看的时候倒正常,编辑的时候发现是乱码的。我记得代码上传到github不是应该都会转成utf-8来着吗?难道是官方bug
哦...搜了一下发现,是不是本地编辑或者git提交设置用别的编码了?比如gb2312?编码不一样上传了转utf-8当然也会出问题

Copy link
Collaborator

@enjoy-binbin enjoy-binbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请尝试通过这种方式来确认编码问题,我觉得建议检查下本地编辑器的编码啥的
image

src/db.c Outdated Show resolved Hide resolved
@artikell artikell force-pushed the feature/lua-annotated branch 2 times, most recently from 6313a96 to dc2e165 Compare November 5, 2022 19:28
@artikell artikell force-pushed the feature/lua-annotated branch 2 times, most recently from 2307705 to c1f25e5 Compare January 28, 2023 02:38
@artikell
Copy link
Author

请尝试通过这种方式来确认编码问题,我觉得建议检查下本地编辑器的编码啥的 image

不好意思,这块一直没处理干净,现在应该是没有编码问题了。

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.

3 participants