From d3b63a9c45ae25409eee8a94bf097361f569805a Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Thu, 10 Mar 2022 07:30:05 +0530
Subject: [PATCH] Cleanup attribute access

---
 kitty/shm.py | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/kitty/shm.py b/kitty/shm.py
index fac2f346f..5f5d866a0 100644
--- a/kitty/shm.py
+++ b/kitty/shm.py
@@ -30,6 +30,9 @@ def make_filename(prefix: str) -> str:
 class SharedMemory:
     _buf: Optional[memoryview] = None
     _fd: int = -1
+    _name: str = ''
+    _mmap: Optional[mmap.mmap] = None
+    _size: int = 0
 
     def __init__(
             self, name: Optional[str] = None, create: bool = False, size: int = 0, readonly: bool = False,
@@ -55,7 +58,6 @@ class SharedMemory:
                     self._fd = shm_open(name, flags, mode)
                 except FileExistsError:
                     continue
-                self._name = name
                 break
         else:
             self._fd = shm_open(name, flags)
@@ -70,7 +72,7 @@ class SharedMemory:
             self.unlink()
             raise
 
-        self.size = size
+        self._size = size
         self._buf = memoryview(self._mmap)
 
     def __del__(self) -> None:
@@ -85,6 +87,10 @@ class SharedMemory:
     def __exit__(self, *a: object) -> None:
         self.close()
 
+    @property
+    def size(self) -> int:
+        return self._size
+
     @property
     def name(self) -> str:
         return self._name
@@ -108,8 +114,9 @@ class SharedMemory:
         if self._buf is not None:
             self._buf.release()
             self._buf = None
-        if getattr(self, '_mmap', None) is not None:
+        if self._mmap is not None:
             self._mmap.close()
+            self._mmap = None
         if self._fd >= 0:
             os.close(self._fd)
             self._fd = -1