AK: Remove Optional::operator bool()

This was causing some obvious-in-hindsight but hard to spot bugs where
we'd implicitly convert the bool to an integer type and carry on with
the number 1 instead of the actual value().

Author: awesomekling

AK/Bitmap.h

@@ -165,14 +165,14 @@ class Bitmap {
         if (!first_index.has_value())
             return {};
 
-        size_t free_region_start = first_index;
+        size_t free_region_start = first_index.value();
         size_t free_region_size = 1;
 
         size_t max_region_start = free_region_start;
         size_t max_region_size = free_region_size;
 
         // Let's try and find the best fit possible
-        for (size_t j = first_index + 1; j < m_size && free_region_size < max_length; j++) {
+        for (size_t j = first_index.value() + 1; j < m_size && free_region_size < max_length; j++) {
             if (!get(j)) {
                 if (free_region_size == 0)
                     free_region_start = j;

AK/Optional.h

@@ -150,9 +150,6 @@ class CONSUMABLE(unknown) alignas(T) Optional {
         return fallback;
     }
 
-    SET_TYPESTATE(consumed)
-    operator bool() const { return m_has_value; }
-
 private:
     // Call when we don't want to alter the consume state
     const T& value_without_consume_state() const

Kernel/FileSystem/Ext2FileSystem.cpp

@@ -1076,9 +1076,9 @@ Ext2FS::BlockIndex Ext2FS::allocate_block(GroupIndex preferred_group_index)
     auto block_bitmap = Bitmap::wrap(cached_bitmap.buffer.data(), blocks_in_group);
 
     BlockIndex first_block_in_group = (group_index - 1) * blocks_per_group() + first_block_index();
-    int first_unset_bit_index = block_bitmap.find_first_unset();
-    ASSERT(first_unset_bit_index != -1);
-    BlockIndex block_index = (unsigned)first_unset_bit_index + first_block_in_group;
+    auto first_unset_bit_index = block_bitmap.find_first_unset();
+    ASSERT(first_unset_bit_index.has_value());
+    BlockIndex block_index = first_unset_bit_index.value() + first_block_in_group;
     set_block_allocation_state(block_index, true);
     return block_index;
 }

Kernel/FileSystem/ProcFS.cpp

@@ -1228,7 +1228,7 @@ ssize_t ProcFSInode::read_bytes(off_t offset, ssize_t count, u8* buffer, FileDes
     if (!description) {
         generated_data = (*read_callback)(identifier());
     } else {
-        if (!description->generator_cache())
+        if (!description->generator_cache().has_value())
             description->generator_cache() = (*read_callback)(identifier());
         generated_data = description->generator_cache();
     }
@@ -1242,7 +1242,7 @@ ssize_t ProcFSInode::read_bytes(off_t offset, ssize_t count, u8* buffer, FileDes
 
     ssize_t nread = min(static_cast<off_t>(data.value().size() - offset), static_cast<off_t>(count));
     memcpy(buffer, data.value().data() + offset, nread);
-    if (nread == 0 && description && description->generator_cache())
+    if (nread == 0 && description && description->generator_cache().has_value())
         description->generator_cache().clear();
 
     return nread;

Libraries/LibHTML/DOM/Element.cpp

@@ -209,7 +209,7 @@ NonnullRefPtr<StyleProperties> Element::computed_style()
         };
         for (CSS::PropertyID id : box_model_metrics) {
             auto prop = layout_node()->style().property(id);
-            if (prop)
+            if (prop.has_value())
                 properties->set_property(id, prop.value());
         }
     }

Servers/LookupServer/LookupServer.cpp

@@ -117,7 +117,7 @@ void LookupServer::service_client(RefPtr<Core::LocalSocket> socket)
 
     Vector<String> responses;
 
-    if (auto known_host = m_etc_hosts.get(hostname)) {
+    if (auto known_host = m_etc_hosts.get(hostname); known_host.has_value()) {
         responses.append(known_host.value());
     } else if (!hostname.is_empty()) {
         bool did_timeout;