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

プロジェクトファイルに含まれていないソースファイルを追加する #1646

Merged
1 commit merged into from Apr 27, 2021

Conversation

ghost
Copy link

@ghost ghost commented Apr 26, 2021

PR の目的

リポジトリに存在するソースコードのうち、プロジェクトファイルに追加されていないものを追加します。

カテゴリ

  • その他の問題

PR の背景

プロジェクトファイルに「 SShare_History.h 」が含まれていないため、VS 上で探せず不便です。

  1. ソリューションエクスプローラーに SShare_History.h が表示されません
  2. クラスビューに SShare_History 構造体が表示されません。

PR のメリット

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

仕様・動作説明

次のファイルが含まれていないので追加します。

  • sakura_core/recent/SShare_History.h ( 24557f0 にて追加)

sakura_core/basis/_com_raise_error.cpp ( #1533 にて追加)も含まれていませんが追加していません。

PR の影響範囲

テスト内容

テスト1

手順

関連 issue, PR

参考資料

@ghost ghost marked this pull request as draft April 26, 2021 11:06
@sonarqubecloud
Copy link

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

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

@ghost ghost marked this pull request as ready for review April 26, 2021 11:19
@AppVeyorBot
Copy link

Build sakura 1.0.3708 failed (commit 0b043a0280 by @kazasaku)

@AppVeyorBot
Copy link

Build sakura 1.0.3709 completed (commit 09a6de4b58 by @kazasaku)

Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

問題無いと思います。

@berryzplus
Copy link
Contributor

sakura_core/basis/_com_raise_error.cpp ( #1533 にて追加)も含まれていませんが追加していません。

このファイルはMinGW向けWindowsSDKの定義漏れ(?)を補う目的のもので、
visual studioプロジェクトに含めると都合が悪いはず。
追加したらリンカエラーになるんじゃないかな、と。

@ghost
Copy link
Author

ghost commented Apr 27, 2021

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

@ghost ghost merged commit 2e01db2 into sakura-editor:master Apr 27, 2021
@ghost ghost deleted the feature/add_sources_to_project branch April 27, 2021 05:00
This pull request was closed.
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