refactor: clean up gui file argument parsing

src/gui/main.cpp

@@ -9,19 +9,7 @@ int app_main(int argc, char* argv[]) { // NOLINT
 
 	std::vector<std::string> arguments(argv + 1, argv + argc);
 
-	// redundant since add_files checks paths too so i could just pass arguments to it, but id rather keep the logic
-	// here for the clarity
-	std::vector<std::string> passed_file_paths;
-	for (const auto& file_path_str : arguments) {
-		std::filesystem::path file_path(file_path_str);
-
-		if (!file_path.empty() && std::filesystem::exists(file_path))
-			passed_file_paths.push_back(file_path_str);
-	}
-
-	std::thread(tasks::run).detach();
-
-	tasks::add_files(passed_file_paths);
+	std::thread(tasks::run, arguments).detach();
 
 	gui::run();
 

src/gui/tasks.cpp

@@ -5,7 +5,7 @@
 #include "gui/renderer.h"
 #include "gui/ui/ui.h"
 
-void tasks::run() {
+void tasks::run(const std::vector<std::string>& arguments) {
 	blur.initialise(false, true);
 
 	rendering.set_progress_callback([] {
@@ -36,14 +36,16 @@ void tasks::run() {
 		gui::renderer::on_render_finished(render, success);
 	});
 
+	add_files(arguments); // todo: mac packaged app support (& linux? does it work?)
+
 	while (true) {
 		rendering.render_videos();
 	}
 }
 
 void tasks::add_files(const std::vector<std::string>& files) {
 	for (const std::string& path_str : files) {
-		std::filesystem::path path(path_str);
+		std::filesystem::path path = std::filesystem::canonical(path_str);
 		if (path.empty() || !std::filesystem::exists(path))
 			continue;
 

src/gui/tasks.h

@@ -1,9 +1,7 @@
 #pragma once
 
-#include "common/config.h"
-
 namespace tasks {
-	void run();
+	void run(const std::vector<std::string>& arguments);
 
 	void add_files(const std::vector<std::string>& files);
 }