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

MinGW版CRTでwcsncat_sのC++オーバーロードが未定義になっていることに対策する #1604

Merged

Conversation

berryzplus
Copy link
Contributor

@berryzplus berryzplus commented Mar 21, 2021

PR の目的

MinGWビルドの失敗を解消することが目的です。

カテゴリ

  • ビルド関連
    • MinGW

PR の背景

#1600 で行った修正により、MinGWビルドが失敗するようになりました。

PR のメリット

現在起きているMinGWビルドの失敗が解消します。

PR のデメリット (トレードオフとかあれば)

本質的な解決になっていないような気がします。

仕様・動作説明

MinGW版CRT でセキュア版関数 wcsncat_s の C++ オーバーロードが 定義されていないことが原因です。

この関数には本来、テンプレートを使った C++ 専用のオーバーロードが存在します。
https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/strncat-s-strncat-s-l-wcsncat-s-wcsncat-s-l-mbsncat-s-mbsncat-s-l

PR の影響範囲

テスト内容

ビルド確認のみです。

関連 issue, PR

#1600

参考資料

@berryzplus berryzplus added 🐛bug🦋 ■バグ修正(Something isn't working) 💩degradation🧻🚽 デグレ (前に動いていた機能が動かなくなった) labels Mar 21, 2021
@sonarcloud
Copy link

sonarcloud bot commented Mar 21, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージしちゃいます。

@berryzplus berryzplus merged commit 82950bf into sakura-editor:master Mar 22, 2021
@berryzplus berryzplus deleted the feature/fix_COutlineErlang branch March 22, 2021 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working) 💩degradation🧻🚽 デグレ (前に動いていた機能が動かなくなった)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants