From bb92162dba872e1d880f7b314396537e4e0a8832 Mon Sep 17 00:00:00 2001
From: Peter Evans <pevans@activecampaign.com>
Date: Tue, 12 Dec 2017 14:45:48 -0600
Subject: [PATCH] Zero, not NULL.

Not that there's a practical difference, but we should be clear that
we're zeroing out memory because its contents are vm_8bit values, not
pointers.
---
 src/vm_segment.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/vm_segment.c b/src/vm_segment.c
index f2aed18..620ef7f 100644
--- a/src/vm_segment.c
+++ b/src/vm_segment.c
@@ -38,9 +38,9 @@ vm_segment_create(size_t size)
         return NULL;
     }
 
-    // We should NULL out memory and make explicit that any new segment
+    // We should zero out memory and make explicit that any new segment
     // begins life in that state.
-    memset(segment->memory, (int)NULL, sizeof(vm_8bit));
+    memset(segment->memory, 0, sizeof(vm_8bit));
 
     segment->read_table = malloc(sizeof(vm_segment_read_fn) * size);
     if (segment->read_table == NULL) {