[applib] Make run take args as a SpanView

src/olympic/applib/applib.cpp

@@ -41,8 +41,7 @@ ox::Error run(
 		ox::StringView project,
 		ox::StringView appName,
 		ox::StringView projectDataDir,
-		int argc,
+		ox::SpanView<char const*> argv) noexcept;
-		char const**argv) noexcept;
 
 }
 
@@ -65,7 +64,11 @@ int main(int argc, char const**argv) {
 #if OLYMPIC_LOAD_STUDIO_MODULES
 	OLYMPIC_PROJECT_NAMESPACE::registerStudioModules();
 #endif
-	auto const err = olympic::run(OLYMPIC_PROJECT_NAME, OLYMPIC_APP_NAME, OLYMPIC_PROJECT_DATADIR, argc, argv);
+	auto const err = olympic::run(
+			OLYMPIC_PROJECT_NAME,
+			OLYMPIC_APP_NAME,
+			OLYMPIC_PROJECT_DATADIR,
+			{argv, static_cast<size_t>(argc)});
 	oxAssert(err, "Something went wrong...");
 	if (err) {
 		oxErrf("Failure: {}\n", toStr(err));

src/olympic/keel/src/module.cpp

@@ -9,7 +9,9 @@ namespace keel {
 static ox::Vector<Module const*> mods;
 
 void registerModule(Module const*mod) noexcept {
-	mods.emplace_back(mod);
+	if (mod) {
+		mods.emplace_back(mod);
+	}
 }
 
 [[nodiscard]]

src/olympic/keel/src/pack-applib.cpp

@@ -81,8 +81,8 @@ static ox::Error pack(
 	return {};
 }
 
-static ox::Error run(int argc, char const**argv, ox::StringView projectDataDir) noexcept {
+static ox::Error run(ox::SpanView<char const*> argv, ox::StringView projectDataDir) noexcept {
-	ox::ClArgs const args(argc, argv);
+	ox::ClArgs const args(argv);
 	auto const argSrc = args.getString("src", "");
 	auto const argRomBin = args.getString("rom-bin", "");
 	auto const argManifest = args.getString("manifest", "");
@@ -103,9 +103,8 @@ ox::Error run(
 		[[maybe_unused]] ox::StringView project,
 		[[maybe_unused]] ox::StringView appName,
 		ox::StringView projectDataDir,
-		int argc,
+		ox::SpanView<char const*> argv) noexcept {
-		char const**argv) noexcept {
+	return ::run(argv, projectDataDir);
-	return ::run(argc, argv, projectDataDir);
 }
 
 }

src/olympic/studio/applib/src/main.cpp

@@ -52,8 +52,7 @@ static ox::Error runApp(
 static ox::Error run(
 		ox::StringViewCR appName,
 		ox::StringViewCR projectDataDir,
-		int,
+		ox::SpanView<const char*>) {
-		char const**) {
 	// seed UUID generator
 	auto const time = std::time(nullptr);
 	ox::UUID::seedGenerator({
@@ -74,9 +73,8 @@ ox::Error run(
 		ox::StringView project,
 		ox::StringView appName,
 		ox::StringView projectDataDir,
-		int argc,
+		ox::SpanView<char const*> args) noexcept {
-		char const**argv) noexcept {
+	return studio::run(ox::sfmt("{} {}", project, appName), projectDataDir, args);
-	return studio::run(ox::sfmt("{} {}", project, appName), projectDataDir, argc, argv);
 }
 
 }

src/olympic/studio/applib/src/studioapp.cpp

@@ -21,7 +21,9 @@ namespace studio {
 static ox::Vector<studio::Module const*> modules;
 
 void registerModule(studio::Module const*mod) noexcept {
-	modules.emplace_back(mod);
+	if (mod) {
+		modules.emplace_back(mod);
+	}
 }
 
 
@@ -238,18 +240,18 @@ void StudioUI::loadEditorMaker(studio::EditorMaker const&editorMaker) noexcept {
 	}
 }
 
-void StudioUI::loadModule(studio::Module const*mod) noexcept {
+void StudioUI::loadModule(studio::Module const&mod) noexcept {
-	for (auto const&editorMaker : mod->editors(m_sctx)) {
+	for (auto const&editorMaker : mod.editors(m_sctx)) {
 		loadEditorMaker(editorMaker);
 	}
-	for (auto &im : mod->itemMakers(m_sctx)) {
+	for (auto &im : mod.itemMakers(m_sctx)) {
 		m_newMenu.addItemMaker(std::move(im));
 	}
 }
 
 void StudioUI::loadModules() noexcept {
 	for (auto const mod : modules) {
-		loadModule(mod);
+		loadModule(*mod);
 	}
 }
 

src/olympic/studio/applib/src/studioapp.hpp

@@ -69,7 +69,7 @@ class StudioUI: public ox::SignalHandler {
 
 		void loadEditorMaker(studio::EditorMaker const&editorMaker) noexcept;
 
-		void loadModule(studio::Module const*mod) noexcept;
+		void loadModule(studio::Module const&mod) noexcept;
 
 		void loadModules() noexcept;