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

一部分力所能及的小修正 #70

Closed
wants to merge 9 commits into from
Closed

一部分力所能及的小修正 #70

wants to merge 9 commits into from

Conversation

FurryR
Copy link

@FurryR FurryR commented Oct 14, 2023

内容

主要是关于底层组件的小修正,以及修复在部分平台上行为不正确的问题。

source/include/Utils/Locale.hpp Outdated Show resolved Hide resolved
source/include/Utils/Locale.hpp Outdated Show resolved Hide resolved
source/include/Utils/Locale.hpp Outdated Show resolved Hide resolved
修正了当目标平台不为 Win32 时转换结果异常的问题
@MistEO MistEO requested review from dantmnf and neko-para October 14, 2023 12:36
@FurryR FurryR marked this pull request as ready for review October 14, 2023 12:44
@FurryR
Copy link
Author

FurryR commented Oct 14, 2023

寄了,我环境上跑不起来编译,看来只能到这了,到时候如果还有更新的东西再开别的 pr 吧。

@MistEO
Copy link
Member

MistEO commented Oct 14, 2023

CMakeLists.txt Outdated Show resolved Hide resolved
@MistEO MistEO marked this pull request as draft December 15, 2023 10:07
@MistEO MistEO force-pushed the main branch 2 times, most recently from bedada1 to f0e8d1d Compare December 20, 2023 08:17
@MistEO MistEO force-pushed the main branch 2 times, most recently from 7309441 to 2d698ec Compare February 27, 2024 15:56
@MistEO MistEO requested a review from neko-para April 3, 2024 08:58
@MistEO
Copy link
Member

MistEO commented Apr 3, 2024

我整理了一下你的代码,把冲突解决了,但是单元测试跑不过,请参考 CI 修改~

@neko-para
Copy link
Contributor

这分支咋复活了(

@MistEO MistEO closed this Apr 4, 2024
@neko-para neko-para mentioned this pull request Aug 22, 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.

3 participants