Remove property accessors and None initialization of Sprite.rect, Sprite.image

buildconfig/stubs/pygame/sprite.pyi

@@ -30,27 +30,21 @@ from pygame.typing import Point, RectLike
 # Some sprite functions only need objects with certain attributes, not always a sprite
 class _HasRect(Protocol):
     @property
-    def rect(self) -> Optional[Union[FRect, Rect]]: ...
+    def rect(self) -> Union[FRect, Rect]: ...
 
 # image in addition to rect
 class _HasImageAndRect(_HasRect, Protocol):
     @property
-    def image(self) -> Optional[Surface]: ...
+    def image(self) -> Surface: ...
 
 # mask in addition to rect
 class _HasMaskAndRect(_HasRect, Protocol):
     @property
     def mask(self) -> Mask: ...
 
 class Sprite(_HasImageAndRect):
-    @property
-    def image(self) -> Optional[Surface]: ...
-    @image.setter
-    def image(self, value: Optional[Surface]) -> None: ...
-    @property
-    def rect(self) -> Optional[Union[FRect, Rect]]: ...
-    @rect.setter
-    def rect(self, value: Optional[Union[FRect, Rect]]) -> None: ...
+    image: Surface  # Uses attribute annotation instead of property to satisfy stubtest
+    rect: Union[FRect, Rect]  # Same as image, see above
     @property
     def layer(self) -> int: ...
     @layer.setter
@@ -68,7 +62,7 @@ class Sprite(_HasImageAndRect):
 class DirtySprite(Sprite):
     dirty: int
     blendmode: int
-    source_rect: Union[FRect, Rect]
+    source_rect: Optional[Union[FRect, Rect]]
     visible: int
     _layer: int
 

src_py/sprite.py

@@ -85,12 +85,13 @@
 # specialized cases.
 
 import types
-from typing import Optional
+from typing import Union
 from warnings import warn
 
 import pygame
 from pygame.mask import from_surface
-from pygame.rect import Rect
+from pygame.rect import FRect, Rect
+from pygame.surface import Surface
 from pygame.time import get_ticks
 
 
@@ -109,29 +110,14 @@ class Sprite:
 
     """
 
+    image: Surface
+    rect: Union[Rect, FRect]
+
     def __init__(self, *groups):
         self.__g = {}  # The groups the sprite is in
-        self.__image: Optional[pygame.surface.Surface] = None
-        self.__rect: Optional[pygame.rect.Rect] = None
         if groups:
             self.add(*groups)
 
-    @property
-    def image(self):
-        return self.__image
-
-    @image.setter
-    def image(self, value: Optional[pygame.surface.Surface]):
-        self.__image = value
-
-    @property
-    def rect(self):
-        return self.__rect
-
-    @rect.setter
-    def rect(self, value: Optional[pygame.rect.Rect]):
-        self.__rect = value
-
     def add(self, *groups):
         """add the sprite to groups