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

_com_raise_errorの独自実装を除去してMinGWビルドのエラーを解消します #1831

Merged

Conversation

sanomari
Copy link
Contributor

@sanomari sanomari commented May 2, 2022

PR の目的

タイトル通りです。

カテゴリ

  • その他の問題

PR の背景

Azure PipelinesでMinGWによるビルドが失敗していますが内容的にこのPRの変更が原因では無いと思います。
Originally posted by @beru in #1830 (review)

というコメントがあったので調べてみました。

エラー原因は、_com_raise_error 関数の再定義でした。

PR のメリット

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

仕様・動作説明

CErrorInfoを実装したとき(#1533)、当時MinGWに未実装だったSDK関数 _com_raise_error を独自に定義して利用するようにしていたんですが、最新のMinGWには _com_raise_error が実装されたようです。

C:/msys64/mingw64/include/comdef.h:166:20: note: 'void _com_raise_error(HRESULT, IErrorInfo*)' previously defined here
  166 | inline void WINAPI _com_raise_error(HRESULT hr, IErrorInfo *perrinfo = 0) {
      |                    ^~~~~~~~~~~~~~~~

詳細なエラーログは こっち

PR の影響範囲

  • MinGWビルド版に影響する変更です。
  • 現行版のMinGW64でビルドできるようになります。
  • 古いMinGW64でビルドできなくなります。
  • 機能的な影響はないと考えられます。

テスト内容

テスト1

手順

関連 issue, PR

参考資料

MinGWに未実装のSDK関数を独自に定義して利用していましたが、
C:/msys64/mingw64/include/comdef.hにインライン実装されるようになったので独自定義を削除します。
@sonarcloud
Copy link

sonarcloud bot commented May 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@AppVeyorBot
Copy link

Build sakura 1.0.4105 completed (commit c51785c51a by @sanomari)

@sanomari sanomari marked this pull request as ready for review May 2, 2022 04:54
@sanomari sanomari merged commit 9d32b54 into sakura-editor:master May 2, 2022
@sanomari sanomari deleted the feature/remove_com_raise_error branch May 2, 2022 06:43
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