diff options
author | Andreas Kling <kling@serenityos.org> | 2021-11-10 11:55:37 +0100 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-11-10 21:58:58 +0100 |
commit | 88b6428c25ea046a4bb19bb6f3f68dd4f1439539 (patch) | |
tree | 86eca67f4ffc83d5387590b0d502ecce7cc07b91 /Kernel/Memory/AddressSpace.cpp | |
parent | cd49f30bea734feb9ac46d637e2ed3439e47e3c3 (diff) | |
download | serenity-88b6428c25ea046a4bb19bb6f3f68dd4f1439539.zip |
AK: Make Vector::try_* functions return ErrorOr<void>
Instead of signalling allocation failure with a bool return value
(false), we now use ErrorOr<void> and return ENOMEM as appropriate.
This allows us to use TRY() and MUST() with Vector. :^)
Diffstat (limited to 'Kernel/Memory/AddressSpace.cpp')
-rw-r--r-- | Kernel/Memory/AddressSpace.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/Kernel/Memory/AddressSpace.cpp b/Kernel/Memory/AddressSpace.cpp index f0ce03dc24..6f3e54d313 100644 --- a/Kernel/Memory/AddressSpace.cpp +++ b/Kernel/Memory/AddressSpace.cpp @@ -110,8 +110,7 @@ ErrorOr<void> AddressSpace::unmap_mmap_range(VirtualAddress addr, size_t size) // Otherwise, split the regions and collect them for future mapping. auto split_regions = TRY(try_split_region_around_range(*region, range_to_unmap)); - if (new_regions.try_extend(split_regions)) - return ENOMEM; + TRY(new_regions.try_extend(split_regions)); } // Give back any unwanted VM to the range allocator. |