Skip to content

Commit 29b8da6

Browse files
committed
docs(probe): note mcpp::process::run_capture for new process invocations
The local run_capture() in probe.cppm returns std::expected<std::string, DetectError>, which differs from mcpp::process::RunResult. Existing callers are left unchanged; the comment steers new code toward the centralised process module.
1 parent b87176f commit 29b8da6

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

src/toolchain/probe.cppm

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,11 @@
11
// mcpp.toolchain.probe - common compiler probing helpers.
2+
//
3+
// NOTE: This file contains its own run_capture() helper that returns
4+
// std::expected<std::string, DetectError> — a different signature from
5+
// mcpp::process::run_capture() (which returns RunResult). Do NOT migrate
6+
// existing callers here without care. For new process invocations that do
7+
// not need DetectError propagation, prefer mcpp::process::run_capture from
8+
// the mcpp.process module.
29

310
module;
411
#include <cstdio> // popen, pclose, fgets, FILE

0 commit comments

Comments
 (0)