Kernel: Clean up the page fault handling code a bit

Not using "else" after "return" unnests the code and makes it easier to
follow. Also use an enum for the two different page fault types.

1 files changed, 7 insertions, 0 deletions

diff --git a/Kernel/Arch/i386/CPU.h b/Kernel/Arch/i386/CPU.h
index a8386ace93..9c12fe9cd3 100644
--- a/Kernel/Arch/i386/CPU.h
+++ b/Kernel/Arch/i386/CPU.h
@@ -303,9 +303,16 @@ public:
     {
     }
 
+    enum class Type {
+        PageNotPresent,
+        ProtectionViolation,
+    };
+
     VirtualAddress vaddr() const { return m_vaddr; }
     u16 code() const { return m_code; }
 
+    Type type() const { return (Type)(m_code & 1); }
+
     bool is_not_present() const { return (m_code & 1) == PageFaultFlags::NotPresent; }
     bool is_protection_violation() const { return (m_code & 1) == PageFaultFlags::ProtectionViolation; }
     bool is_read() const { return (m_code & 2) == PageFaultFlags::Read; }