MSVC warnings in environment.hpp and default_launcher.hpp
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 88/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- cpp
調査の方向性
レポートに挙げられた2つのファイルから始めてください。v2/environment.hpp の1504行目付近では、pathext のループで p が再度宣言され、外側のローカル変数を隠しています。v2/windows/default_launcher.hpp の397行目付近では、build_command_line_impl が未使用のパラメータ args を受け取っています。MSVC でビルドして、警告 C4456 と C4100 を再現してください。両方の警告が消え、動作を変えずに Windows のテストが引き続き通れば完了です。
索引モデルが issue の本文から書いたものです。
説明
The first one is: "v2\environment.hpp(1504,18): warning C4456: declaration of 'p' hides previous local declaration"
In this code:
for (auto ext : pathext)
{
ec.clear();
BOOST_PROCESS_V2_NAMESPACE::filesystem::path nm(name);
nm.concat(ext.begin(), ext.end());
auto p = pp / nm;
if (detail::is_executable(p, ec) && !ec)
return p;
}
Should be probably changed to p = pp / nm;
The second one is on 2 lines (371, 397), showing only one as they are essentially the same: "v2\windows\default_launcher.hpp(397,39): warning C4100: 'args': unreferenced parameter"
In this code:
template<typename Argv>
static std::wstring build_command_line_impl(
const filesystem::path & pt,
const Argv & argv,
basic_string_view<char> args)
Commenting out the parameter should fix the warning.
- 主要言語
- C++
- スター
- 145
- フォーク
- 151
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
boostorg/process のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
-
v2: BOOST_PROCESS_V2_DISABLE_SIGNALSET does not compile with the signal-based process handle対応中かも @kislinsk が 2 日前に担当しました。 オープン
難易度 3/5 半日 初心者へのやさしさ 62/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
boostorg/process の issue をすべて見る
似ている issue
-
category:port-update
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 2 日以内に返信
-
improvement ui
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ihhub/fheroes2#11062 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
enhancement question
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
DarkStarSystems/pcons#224 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
LizardByte/Sunshine#5953 ·
メンテナーはふだん 1 日以内に返信