Bug 581932 ArrayIndexOutOfBoundsException in ArrayIntCompressed

Check Arrays.parallelSort() Change-Id: Idda2f4288d4d57dfea79e43b7c1a7a09e6609903
eclipse-mat · Jun 12, 2023 · d7d3105 · d7d3105
1 parent e592bfd
commit d7d3105
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/plugins/org.eclipse.mat.parser/src/org/eclipse/mat/parser/index/IndexWriter.java b/plugins/org.eclipse.mat.parser/src/org/eclipse/mat/parser/index/IndexWriter.java
@@ -167,6 +167,7 @@ else if (identifiers == null)
                 collect = null;
             }
             Arrays.parallelSort(identifiers, 0, size);
+            Arrays.sort(identifiers, 0, size); // Debug for bug 581932
         }
 
         public int size()

diff --git a/plugins/org.eclipse.mat.report/src/org/eclipse/mat/collect/ArrayInt.java b/plugins/org.eclipse.mat.report/src/org/eclipse/mat/collect/ArrayInt.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2008, 2018 SAP AG and IBM Corporation.
+ * Copyright (c) 2008, 2023 SAP AG and IBM Corporation.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
  * which accompanies this distribution, and is available at
@@ -218,6 +218,7 @@ public int firstElement()
     public void sort()
     {
         Arrays.parallelSort(elements, 0, size);
+        Arrays.sort(elements, 0, size); // Debug for bug 581932
     }
 
     // //////////////////////////////////////////////////////////////

diff --git a/plugins/org.eclipse.mat.report/src/org/eclipse/mat/collect/ArrayLong.java b/plugins/org.eclipse.mat.report/src/org/eclipse/mat/collect/ArrayLong.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2008, 2018 SAP AG and IBM Corporation.
+ * Copyright (c) 2008, 2023 SAP AG and IBM Corporation.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
  * which accompanies this distribution, and is available at
@@ -216,6 +216,7 @@ public long firstElement()
     public void sort()
     {
         Arrays.parallelSort(elements, 0, size);
+        Arrays.sort(elements, 0, size); // Debug for bug 581932
     }
 
     // //////////////////////////////////////////////////////////////