From bdf0b670755beed6c9a24348174af8dd0d927b94 Mon Sep 17 00:00:00 2001
From: Rob McMullen <feedback@playermissile.com>
Date: Sun, 12 Mar 2017 21:46:24 -0700
Subject: [PATCH] Fixed segment rawdata copy for non-indexed segments

---
 atrcopy/segments.py  |  4 ++--
 test/test_segment.py | 21 ++++++++++++++-------
 2 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/atrcopy/segments.py b/atrcopy/segments.py
index b28fba2..ace40df 100644
--- a/atrcopy/segments.py
+++ b/atrcopy/segments.py
@@ -228,8 +228,8 @@ class SegmentData(object):
             s = self.style.np_data.copy()
             copy = SegmentData(d, s, order=self.order)
         else:
-            d = self.data
-            s = self.style
+            d = self.data.copy()
+            s = self.style.copy()
             start, end = self.byte_bounds_offset()
             copy = SegmentData(d[start:end], s[start:end])
         return copy
diff --git a/test/test_segment.py b/test/test_segment.py
index 7882c53..e619bc6 100644
--- a/test/test_segment.py
+++ b/test/test_segment.py
@@ -38,10 +38,17 @@ class TestSegment1(object):
 
     def test_copy(self):
         for s in self.segments:
-            c = s.rawdata.copy()
-            print c.data.shape, c.is_indexed
-            print c.data.np_data, s.data.np_data
-            assert np.all((c.data - s.data) == 0)
+            d = s.rawdata
+            print "orig:", d.data.shape, d.is_indexed, d.data, id(d.data)
+            c = d.copy()
+            print "copy", c.data.shape, c.is_indexed, c.data, id(c.data)
+            assert c.data.shape == s.data.shape
+            assert id(c) != id(s)
+            assert np.all((c.data[:] - s.data[:]) == 0)
+            c.data[0:100] = 1
+            print d.data
+            print c.data
+            assert not np.all((c.data[:] - s.data[:]) == 0)
 
 
 class TestIndexed(object):
@@ -180,7 +187,7 @@ if __name__ == "__main__":
     t.test_indexed()
     t.test_indexed_sub()
     t.test_interleave()
-    #t = TestSegment1()
-    #t.setup()
-    #t.test_xex()
+    t = TestSegment1()
+    t.setup()
+    t.test_xex()
     t.test_copy()