You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemds.apache.org by mb...@apache.org on 2023/07/19 15:28:46 UTC

[systemds] branch main updated: [MINOR] Fix corrupted test util for compression-component tests

This is an automated email from the ASF dual-hosted git repository.

mboehm7 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/systemds.git


The following commit(s) were added to refs/heads/main by this push:
     new 7151984d14 [MINOR] Fix corrupted test util for compression-component tests
7151984d14 is described below

commit 7151984d14b3d858271cd2efc0c7b585299e9914
Author: Matthias Boehm <mb...@gmail.com>
AuthorDate: Wed Jul 19 17:08:23 2023 +0200

    [MINOR] Fix corrupted test util for compression-component tests
---
 .../sysds/test/component/compress/mapping/MappingTestUtil.java    | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/test/java/org/apache/sysds/test/component/compress/mapping/MappingTestUtil.java b/src/test/java/org/apache/sysds/test/component/compress/mapping/MappingTestUtil.java
index 08a247f726..bcb9703a96 100644
--- a/src/test/java/org/apache/sysds/test/component/compress/mapping/MappingTestUtil.java
+++ b/src/test/java/org/apache/sysds/test/component/compress/mapping/MappingTestUtil.java
@@ -33,31 +33,25 @@ import org.apache.sysds.runtime.compress.colgroup.offset.OffsetFactory;
 import org.apache.sysds.runtime.compress.utils.IntArrayList;
 
 public class MappingTestUtil {
+	@SuppressWarnings("fallthrough")
 	protected static AMapToData[] getAllHigherVersions(AMapToData m) {
 		AMapToData[] ret = new AMapToData[getTypeSize(m.getType())];
 		int idx = 0;
 		switch(m.getType()) {
 			case ZERO:
 				ret[idx++] = MapToFactory.resizeForce(m, MAP_TYPE.ZERO);
-				break;
 			case BIT:
 				ret[idx++] = MapToFactory.resizeForce(m, MAP_TYPE.UBYTE);
-				break;
 			case UBYTE:
 				ret[idx++] = MapToFactory.resizeForce(m, MAP_TYPE.BYTE);
-				break;
 			case BYTE:
 				ret[idx++] = MapToFactory.resizeForce(m, MAP_TYPE.CHAR);
-				break;
 			case CHAR:
 				ret[idx++] = MapToFactory.resizeForce(m, MAP_TYPE.CHAR_BYTE);
-				break;
 			case CHAR_BYTE:
 				ret[idx++] = MapToFactory.resizeForce(m, MAP_TYPE.INT);
-				break;
 			case INT:
 				// none
-				break;
 		}
 		return ret;
 	}