From ae0c8e66d01d84e36ab154faac9e7d136d25619a Mon Sep 17 00:00:00 2001
From: Anthony Geay <anthony.geay@edf.fr>
Date: Wed, 16 Dec 2015 19:37:10 +0100
Subject: [PATCH] Bug correction on computeOffsets2 for empty arrayint. Shame
 on me \!

---
 src/MEDCoupling/MEDCouplingMemArray.cxx       |  2 --
 src/MEDCoupling_Swig/MEDCouplingBasicsTest.py | 13 +++++++++++++
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/MEDCoupling/MEDCouplingMemArray.cxx b/src/MEDCoupling/MEDCouplingMemArray.cxx
index 998ac654e..72d3a1fe3 100644
--- a/src/MEDCoupling/MEDCouplingMemArray.cxx
+++ b/src/MEDCoupling/MEDCouplingMemArray.cxx
@@ -10356,8 +10356,6 @@ void DataArrayInt::computeOffsets2()
     throw INTERP_KERNEL::Exception("DataArrayInt::computeOffsets2 : only single component allowed !");
   int nbOfTuples=getNumberOfTuples();
   int *ret=(int *)malloc((nbOfTuples+1)*sizeof(int));
-  if(nbOfTuples==0)
-    return ;
   const int *work=getConstPointer();
   ret[0]=0;
   for(int i=0;i<nbOfTuples;i++)
diff --git a/src/MEDCoupling_Swig/MEDCouplingBasicsTest.py b/src/MEDCoupling_Swig/MEDCouplingBasicsTest.py
index 18118dc75..8e19f49e6 100644
--- a/src/MEDCoupling_Swig/MEDCouplingBasicsTest.py
+++ b/src/MEDCoupling_Swig/MEDCouplingBasicsTest.py
@@ -16770,6 +16770,19 @@ class MEDCouplingBasicsTest(unittest.TestCase):
         self.assertTrue(not "mismatch" in m.__repr__())# bug was here !
         pass
 
+    def testSwig2BugComputeOffsets1(self):
+        """Non regression test. computeOffsets2 on empty array must return 0."""
+        d=DataArrayInt([3])
+        d.computeOffsets2()
+        self.assertTrue(d.isEqual(DataArrayInt([0,3])))
+        d=DataArrayInt([])
+        d.computeOffsets()
+        self.assertTrue(d.isEqual(DataArrayInt([])))
+        d=DataArrayInt([])
+        d.computeOffsets2()
+        self.assertTrue(d.isEqual(DataArrayInt([0]))) # <- bug was here
+        pass
+
     pass
 
 if __name__ == '__main__':
-- 
2.39.2