Userland+AK: Stop using getopt() for ArgsParser

This commit moves the implementation of getopt into AK, and converts its
API to understand and use StringView instead of char*.
Everything else is caught in the crossfire of making
Option::accept_value() take a StringView instead of a char const*.

With this, we must now pass a Span<StringView> to ArgsParser::parse(),
applications using LibMain are unaffected, but anything not using that
or taking its own argc/argv has to construct a Vector<StringView> for
this method.

1 files changed, 2 insertions, 2 deletions

diff --git a/Userland/Utilities/gron.cpp b/Userland/Utilities/gron.cpp
index e30846e206..12dc449a15 100644
--- a/Userland/Utilities/gron.cpp
+++ b/Userland/Utilities/gron.cpp
@@ -45,8 +45,8 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
         "monochrome",
         'm',
         nullptr,
-        [](char const* s) {
-            VERIFY(s == nullptr);
+        [](StringView s) {
+            VERIFY(s.is_empty());
             use_color = false;
             return true;
         },