Use ARRAYSZ() instead of sizeof division, for readability.

Also, it's not vulnerable to changes to the underlying types.

1 files changed, 3 insertions, 3 deletions

diff --git a/crawl-ref/source/bitary.h b/crawl-ref/source/bitary.h
index 291ca4bb88..b01f9ddff5 100644
--- a/crawl-ref/source/bitary.h
+++ b/crawl-ref/source/bitary.h
@@ -41,7 +41,7 @@ protected:
 public:
     void reset()
     {
-        for (unsigned int i = 0; i < sizeof(data) / sizeof(unsigned long); i++)
+        for (unsigned int i = 0; i < ARRAYSZ(data); i++)
             data[i] = 0;
     }
 
@@ -79,14 +79,14 @@ public:
 
     inline FixedBitArray<SIZE>& operator|=(const FixedBitArray<SIZE>&x)
     {
-        for (unsigned int i = 0; i < sizeof(data) / sizeof(unsigned long); i++)
+        for (unsigned int i = 0; i < ARRAYSZ(data); i++)
             data[i] |= x.data[i];
         return *this;
     }
 
     inline FixedBitArray<SIZE>& operator&=(const FixedBitArray<SIZE>&x)
     {
-        for (unsigned int i = 0; i < sizeof(data) / sizeof(unsigned long); i++)
+        for (unsigned int i = 0; i < ARRAYSZ(data); i++)
             data[i] &= x.data[i];
         return *this;
     }