ToolArgs include no default strings

#fix

Author: alandefreitas

src/lib/Lib/Config.cpp

@@ -150,7 +150,8 @@ struct PublicSettingsVisitor {
         std::string& value,
         ReferenceDirectories const& dirs,
         PublicSettings::OptionProperties const& opts,
-        bool const usingDefault) {
+        bool const usingDefault)
+    {
         // If the path is not absolute, we need to expand it
         if (!files::isAbsolute(value))
         {
@@ -258,7 +259,7 @@ struct PublicSettingsVisitor {
     {
         for (auto& value : values)
         {
-            MRDOCS_DECL(normalizeStringPath(self, name, value, dirs, opts, usingDefault));
+            MRDOCS_TRY(normalizeStringPath(self, name, value, dirs, opts, usingDefault));
         }
 
         // Move command line sink values to appropriate destinations

src/tool/ToolMain.cpp

@@ -19,11 +19,11 @@
 #include <llvm/Support/PrettyStackTrace.h>
 #include <llvm/Support/Signals.h>
 #include <cstdlib>
+#include <ranges>
 
 extern int main(int argc, char const** argv);
 
-namespace clang {
-namespace mrdocs {
+namespace clang::mrdocs {
 
 extern
 int
@@ -47,7 +47,7 @@ print_version(llvm::raw_ostream& os)
        << "\n";
 }
 
-Expected<std::pair<std::string, ReferenceDirectories>>
+Expected<ReferenceDirectories>
 getReferenceDirectories(std::string const& execPath)
 {
     ReferenceDirectories dirs;
@@ -58,36 +58,37 @@ getReferenceDirectories(std::string const& execPath)
         return Unexpected(formatError("Unable to determine current working directory: {}", ec.message()));
     }
     dirs.cwd = std::string(cwd.data(), cwd.size());
+    return dirs;
+}
+
+Expected<std::string>
+getConfigPath(ReferenceDirectories const& dirs)
+{
     std::string configPath;
-    if (toolArgs.config.getValue() != "")
+    auto cmdLineFilenames = std::ranges::views::transform(
+        toolArgs.cmdLineInputs, files::getFileName);
+    if (!toolArgs.config.getValue().empty())
     {
+        // From explicit --config argument
         configPath = toolArgs.config.getValue();
     }
-    else
+    else if (auto const it = std::ranges::find(cmdLineFilenames, "mrdocs.yml");
+             it != cmdLineFilenames.end())
     {
-        llvm::cl::list<std::string>& inputs = toolArgs.cmdLineInputs;
-        for (auto& input: inputs)
-        {
-            if (files::getFileName(input) == "mrdocs.yml")
-            {
-                configPath = input;
-                break;
-            }
-        }
+        // From implicit command line inputs
+        configPath = *(it.base());
     }
-    if (configPath.empty())
+    else if (files::exists("./mrdocs.yml"))
     {
-        if (files::exists("./mrdocs.yml"))
-        {
-            configPath = "./mrdocs.yml";
-        }
+        // From current directory
+        configPath = "./mrdocs.yml";
     }
-    if (configPath.empty())
+    else
     {
         return Unexpected(formatError("The config path is missing"));
     }
     configPath = files::makeAbsolute(configPath, dirs.cwd);
-    return std::make_pair(configPath, dirs);
+    return configPath;
 }
 
 int
@@ -129,7 +130,15 @@ mrdocs_main(int argc, char const** argv)
         report::fatal("Failed to determine reference directories: {}", res.error().message());
         return EXIT_FAILURE;
     }
-    auto [configPath, dirs] = *res;
+    auto dirs = *std::move(res);
+
+    auto expConfigPath = getConfigPath(dirs);
+    if (!expConfigPath)
+    {
+        report::fatal("Failed to determine config path: {}", expConfigPath.error().message());
+        return EXIT_FAILURE;
+    }
+    auto configPath = *std::move(expConfigPath);
 
     // Generate
     auto exp = DoGenerateAction(configPath, dirs, argv);
@@ -156,8 +165,7 @@ reportUnhandledException(
     sys::PrintStackTrace(llvm::errs());
 }
 
-} // mrdocs
-} // clang
+} // clang::mrdocs
 
 int
 main(int argc, char const** argv)

util/generate-config-info.py

@@ -839,11 +839,11 @@ def generate_toolargs_final_option_initializer(option, category_str, parents=Non
     if option['command-line-sink']:
         constructor_args.append('llvm::cl::Sink')
     if 'default' in option:
-        if option["type"] in ['string', 'enum']:
-            constructor_args.append(f'llvm::cl::init("{option["default"]}")')
-        elif option["type"] in ['path', 'file-path', 'dir-path']:
-            constructor_args.append(f'llvm::cl::init("{remove_reference_dir_from_path(option["default"])}")')
-        elif option['type'] in ['unsigned', 'int']:
+        # if option["type"] in ['string', 'enum']:
+        #     constructor_args.append(f'llvm::cl::init("{option["default"]}")')
+        # elif option["type"] in ['path', 'file-path', 'dir-path']:
+        #     constructor_args.append(f'llvm::cl::init("{remove_reference_dir_from_path(option["default"])}")')
+        if option['type'] in ['unsigned', 'int']:
             constructor_args.append(f'llvm::cl::init({option["default"]})')
         elif option['type'] == 'bool':
             bool_str = 'true' if option['default'] else 'false'