diff --git a/Meta/Lagom/CMakeLists.txt b/Meta/Lagom/CMakeLists.txt index 3ba6966dc7..d392d51f15 100644 --- a/Meta/Lagom/CMakeLists.txt +++ b/Meta/Lagom/CMakeLists.txt @@ -513,7 +513,7 @@ if (BUILD_LAGOM) add_executable(markdown-check_lagom ../../Userland/Utilities/markdown-check.cpp) set_target_properties(markdown-check_lagom PROPERTIES OUTPUT_NAME markdown-check) - target_link_libraries(markdown-check_lagom LagomMarkdown) + target_link_libraries(markdown-check_lagom LagomMarkdown LagomMain) add_executable(ntpquery_lagom ../../Userland/Utilities/ntpquery.cpp) set_target_properties(ntpquery_lagom PROPERTIES OUTPUT_NAME ntpquery) diff --git a/Userland/Utilities/CMakeLists.txt b/Userland/Utilities/CMakeLists.txt index e66375f4ba..3660433000 100644 --- a/Userland/Utilities/CMakeLists.txt +++ b/Userland/Utilities/CMakeLists.txt @@ -137,7 +137,7 @@ target_link_libraries(lsof LibMain) target_link_libraries(lspci LibPCIDB LibMain) target_link_libraries(lsusb LibUSBDB LibMain) target_link_libraries(man LibMarkdown LibMain) -target_link_libraries(markdown-check LibMarkdown) +target_link_libraries(markdown-check LibMarkdown LibMain) target_link_libraries(matroska LibMain LibVideo) target_link_libraries(md LibMarkdown LibMain) target_link_libraries(mkdir LibMain) diff --git a/Userland/Utilities/markdown-check.cpp b/Userland/Utilities/markdown-check.cpp index 50020c83c0..7a9c17cf16 100644 --- a/Userland/Utilities/markdown-check.cpp +++ b/Userland/Utilities/markdown-check.cpp @@ -15,12 +15,11 @@ #include #include #include -#include #include -#include #include #include #include +#include #include #include #include @@ -223,25 +222,25 @@ RecursionDecision MarkdownLinkage::visit(Markdown::Text::LinkNode const& link_no return RecursionDecision::Recurse; } -int main(int argc, char** argv) +ErrorOr serenity_main(Main::Arguments arguments) { - if (argc < 2) { + if (arguments.strings.size() < 2) { // Technically it is valid to call this program with zero markdown files: When there are // no files, there are no dead links. However, any such usage is probably erroneous. - warnln("Usage: {} Foo.md Bar.md ...", argv[0]); + warnln("Usage: {} Foo.md Bar.md ...", arguments.strings[0]); // E.g.: find AK/ Base/ Documentation/ Kernel/ Meta/ Ports/ Tests/ Userland/ -name '*.md' -print0 | xargs -0 ./MarkdownCheck return 1; } outln("Reading and parsing Markdown files ..."); HashMap files; - for (int i = 1; i < argc; ++i) { - auto path = argv[i]; + for (size_t i = 1; i < arguments.strings.size(); ++i) { + auto path = arguments.strings[i]; auto file_or_error = Core::File::open(path, Core::OpenMode::ReadOnly); if (file_or_error.is_error()) { warnln("Failed to read {}: {}", path, file_or_error.error()); // Since this should never happen anyway, fail early. - return 1; + return file_or_error.release_error(); } auto file = file_or_error.release_value(); auto content_buffer = file->read_all();