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, 1 insertions, 1 deletions

diff --git a/Userland/Services/DHCPClient/DHCPv4Client.cpp b/Userland/Services/DHCPClient/DHCPv4Client.cpp
index 7b9d5f2728..d17b6c5d0f 100644
--- a/Userland/Services/DHCPClient/DHCPv4Client.cpp
+++ b/Userland/Services/DHCPClient/DHCPv4Client.cpp
@@ -169,7 +169,7 @@ ErrorOr<DHCPv4Client::Interfaces> DHCPv4Client::get_discoverable_interfaces()
     auto file_contents = file->read_all();
     auto json = JsonValue::from_string(file_contents);
 
-    if (!json.has_value() || !json.value().is_array()) {
+    if (json.is_error() || !json.value().is_array()) {
         dbgln("Error: No network adapters available");
         return Error::from_string_literal("No network adapters available"sv);
     }