commit 910cf4da9b6c1909a466b9e17f5f90676b54453b
author Maciej Żenczykowski <maze@google.com> Sun Aug 20 12:02:35 2023 +0000
committer Maciej Żenczykowski <maze@google.com> Sun Aug 20 12:03:26 2023 +0000
tree 8e5353bad9895cf3ea66b8bd2b116738d8093902
parent 10fe61b52008b5f0e1641e53d7fceebe1676a6d8

BpfMap - slightly simplify isEmpty()

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I9b93224ec2d4ff717d5e39c3ad50e63ac73f5f7c

staticlibs/native/bpf_headers/include/bpf/BpfMap.h

diff --git a/staticlibs/native/bpf_headers/include/bpf/BpfMap.h b/staticlibs/native/bpf_headers/include/bpf/BpfMap.h
index 51e6d16..1ddcfe6 100644
--- a/staticlibs/native/bpf_headers/include/bpf/BpfMap.h
+++ b/staticlibs/native/bpf_headers/include/bpf/BpfMap.h
@@ -235,12 +235,9 @@
 
     base::Result<bool> isEmpty() const {
         auto key = getFirstKey();
-        if (!key.ok()) {
-            // Return error code ENOENT means the map is empty
-            if (key.error().code() == ENOENT) return true;
-            return key.error();
-        }
-        return false;
+        if (key.ok()) return false;
+        if (key.error().code() == ENOENT) return true;
+        return key.error();
     }
 
   private: