AK: Make JSON parser return ErrorOr<JsonValue> (instead of Optional)

Also add slightly richer parse errors now that we can include a string
literal with returned errors.

This will allow us to use TRY() when working with JSON data.

1 files changed, 2 insertions, 3 deletions

diff --git a/Userland/Applications/SpaceAnalyzer/main.cpp b/Userland/Applications/SpaceAnalyzer/main.cpp
index 31b5fc1264..ec41f9d4a5 100644
--- a/Userland/Applications/SpaceAnalyzer/main.cpp
+++ b/Userland/Applications/SpaceAnalyzer/main.cpp
@@ -84,10 +84,9 @@ static void fill_mounts(Vector<MountInfo>& output)
     }
 
     auto content = file->read_all();
-    auto json = JsonValue::from_string(content);
-    VERIFY(json.has_value());
+    auto json = JsonValue::from_string(content).release_value_but_fixme_should_propagate_errors();
 
-    json.value().as_array().for_each([&output](auto& value) {
+    json.as_array().for_each([&output](auto& value) {
         auto& filesystem_object = value.as_object();
         MountInfo mount_info;
         mount_info.mount_point = filesystem_object.get("mount_point").to_string();