lib.memory: remove Memory.Init.depth.

This attribute is fully redundant with `.__len__()`, and is out of place
on a `list`-like container like `Memory.Init`.

The `.shape` attribute, however, provides a unique function.

Author: whitequark

amaranth/lib/memory.py

@@ -88,11 +88,6 @@ def __init__(self, elems, *, shape, depth):
         def shape(self):
             return self._shape
 
-        # TODO: redundant with __len__
-        @property
-        def depth(self):
-            return self._depth
-
         def __getitem__(self, index):
             return self._elems[index]
 

tests/test_lib_memory.py

@@ -285,7 +285,7 @@ def test_constructor(self):
         self.assertEqual(m.shape, 8)
         self.assertEqual(m.depth, 4)
         self.assertEqual(m.init.shape, 8)
-        self.assertEqual(m.init.depth, 4)
+        self.assertEqual(len(m.init), 4)
         self.assertEqual(m.attrs, {})
         self.assertIsInstance(m.init, memory.Memory.Init)
         self.assertEqual(list(m.init), [1, 2, 3, 0])