Skip to content

Use std::filesystem::path instead of string for file paths #5627

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.

Sign up for GitHub

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

Closed
dschuff wants to merge 13 commits into from
Closed

Use std::filesystem::path instead of string for file paths #5627

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
63f61da
Use std::filesystem::path instead of string for file paths
dschuff Apr 5, 2023
afc9914
clang-format
dschuff Apr 5, 2023
7158017
temporarily bump OSX target
dschuff Apr 5, 2023
cac02d4
convert path usage in tools
dschuff Apr 5, 2023
e536457
fix empty check
dschuff Apr 5, 2023
4034ce8
Merge branch 'main' into filesystem
dschuff Apr 5, 2023
d361806
fix bad path append
dschuff Apr 6, 2023
80d2528
review suggestion
dschuff Apr 6, 2023
3b5f5cd
use binDir path in path.cpp
dschuff Apr 6, 2023
6d0deda
use more paths, explicit conversions to string
dschuff Apr 6, 2023
002e647
clang-format
dschuff Apr 6, 2023
2fd367b
use fs abstraction
dschuff Apr 6, 2023
7695817
exp
dschuff Apr 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix empty check
  • Loading branch information
@dschuff
dschuff committed Apr 5, 2023
commit e536457819cc8e63bd8987f785e0bb792c8ecd68
2 changes: 1 addition & 1 deletion src/tools/wasm-metadce.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -512,7 +512,7 @@ int main(int argc, const char* argv[]) {
});
options.parse(argc, argv);

if (!graphFile.empty()) {
if (graphFile.empty()) {
Fatal() << "no graph file provided.";
}

Expand Down