diff options
| author | Sam Atkins <atkinssj@serenityos.org> | 2022-02-06 13:33:42 +0000 |
|---|---|---|
| committer | Tim Flynn <trflynn89@pm.me> | 2022-02-16 19:49:41 -0500 |
| commit | 8260135d4d6cc50d15b459feb6010a989fcb7f5b (patch) | |
| tree | 8211a9b8b29b749270c6d71f4faf6073e1a382cb /Userland/DevTools | |
| parent | 1a4dd47d5f691f77f5a11dbb5cdd8d58b2336245 (diff) | |
| download | serenity-8260135d4d6cc50d15b459feb6010a989fcb7f5b.zip | |
LibCore+Everywhere: Return ErrorOr from ConfigFile factory methods
I've attempted to handle the errors gracefully where it was clear how to
do so, and simple, but a lot of this was just adding
`release_value_but_fixme_should_propagate_errors()` in places.
Diffstat (limited to 'Userland/DevTools')
| -rw-r--r-- | Userland/DevTools/HackStudio/ProjectTemplate.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Userland/DevTools/HackStudio/ProjectTemplate.cpp b/Userland/DevTools/HackStudio/ProjectTemplate.cpp index 34a2523624..2f823c9143 100644 --- a/Userland/DevTools/HackStudio/ProjectTemplate.cpp +++ b/Userland/DevTools/HackStudio/ProjectTemplate.cpp @@ -30,7 +30,10 @@ ProjectTemplate::ProjectTemplate(const String& id, const String& name, const Str RefPtr<ProjectTemplate> ProjectTemplate::load_from_manifest(const String& manifest_path) { - auto config = Core::ConfigFile::open(manifest_path); + auto maybe_config = Core::ConfigFile::open(manifest_path); + if (maybe_config.is_error()) + return {}; + auto config = maybe_config.release_value(); if (!config->has_group("HackStudioTemplate") || !config->has_key("HackStudioTemplate", "Name") |