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 のコンパイルエラーを修正する #1299

Merged
merged 1 commit into from
May 11, 2020

Conversation

k-takata
Copy link
Member

PR の目的

(Close #1298)
#1298 のコンパイルエラーを修正する。

カテゴリ

  • 不具合修正
  • ビルド関連
    • Azure Pipelines
    • ローカルビルド

PR の背景

GCC 10.1 を使うとコンパイルエラーになる。

仕様・動作説明

どうやら GCC では #if でコンパイル対象とならない部分も、文法チェックを行うようになった模様。
#if 0 で無効化されている部分も、正しくコメントを使うようにする。

テスト内容

CI が通ることを確認する。

PR の影響範囲

#if 0 で無効化されている部分の修正のため、MinGW ビルド以外には影響なし。

関連 issue, PR

#1298

It seems that GCC 10.1 reports errors inside #if 0 ... #endif.
Use comments properly even inside #if.
@AppVeyorBot
Copy link

Build sakura 1.0.2809 completed (commit 951233cc9b by @k-takata)

@k-takata k-takata marked this pull request as ready for review May 11, 2020 05:07
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

さすがっす!問題ないように見えました。

@k-takata k-takata merged commit 5db7c8e into sakura-editor:master May 11, 2020
@k-takata k-takata deleted the fix-mingw-error branch May 11, 2020 05:30
@m-tmatma
Copy link
Member

さすがっす!問題ないように見えました。

「さすがっす」といえば
https://www.youtube.com/watch?v=qZPQSje53Dg

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
MinGW のコンパイルエラーを修正する
@k-takata k-takata added the MinGW MinGW label Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MinGW MinGW
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(master で) Azure Pipelines の MinGW が失敗している
4 participants