Merge branch 'release/2.17.3'

doc/com.minres.scviewer.doc/pom.xml

@@ -7,7 +7,7 @@
 	<parent>
 		<groupId>com.minres.scviewer</groupId>
 		<artifactId>com.minres.scviewer.parent</artifactId>
-		<version>2.17.2</version>
+		<version>2.17.3</version>
 		<relativePath>../..</relativePath>
 	</parent>
 

features/com.minres.scviewer.database.feature/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>3.0.0-SNAPSHOT</version>

features/com.minres.scviewer.e4.feature/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>1.1.0-SNAPSHOT</version>

features/com.minres.scviewer.e4.help.feature/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>1.0.0-SNAPSHOT</version>

features/com.minres.scviewer.e4.platform.feature/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>1.0.0-SNAPSHOT</version>

features/com.minres.scviewer.feature/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>1.1.0-SNAPSHOT</version>

features/com.minres.scviewer.ui.feature/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>1.1.0-SNAPSHOT</version>

plugins/com.minres.scviewer.database.ftr/pom.xml

@@ -6,7 +6,7 @@
 	<parent>
 		<groupId>com.minres.scviewer</groupId>
 		<artifactId>com.minres.scviewer.parent</artifactId>
-		<version>2.17.2</version>
+		<version>2.17.3</version>
 		<relativePath>../..</relativePath>
 	</parent>
 	<packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.database.ftr/src/com/minres/scviewer/database/ftr/FtrDbLoader.java

@@ -49,7 +49,7 @@
 public class FtrDbLoader implements IWaveformDbLoader {
 
 	enum FileType { NONE, PLAIN, GZIP, LZ4};
-
+	
 	/** The max time. */
 	private Long maxTime = 0L;
 
@@ -243,29 +243,31 @@ public List<? extends ITxAttribute> parseAtrributes(byte[] chunk, long blockOffs
 					assert(sz==3);
 					long name_id = cborDecoder.readInt();
 					long type_id = cborDecoder.readInt();
+					DataType type = DataType.values()[(int)type_id];
 					String attrName = strDict.get((int)name_id);
-					TxAttributeType attrType = getOrAddAttributeType(tag, type_id, attrName);
-					switch((int)type_id) {
-					case 0: // BOOLEAN
+					TxAttributeType attrType = getOrAddAttributeType(tag, type, attrName);
+					switch(type) {
+					case BOOLEAN:
 						ITxAttribute b = new TxAttribute(attrType, cborDecoder.readBoolean()?"True":"False");
 						ret.add(b);
 						break;
-					case 2: // INTEGER
-					case 3: // UNSIGNED
-					case 10: // POINTER
+					case INTEGER:
+					case UNSIGNED:
+					case POINTER:
+					case TIME:
 						ITxAttribute a = new TxAttribute(attrType, String.valueOf(cborDecoder.readInt()));
 						ret.add(a);
 						break;
-					case 4: // FLOATING_POINT_NUMBER
-					case 7: // FIXED_POINT_INTEGER
-					case 8: // UNSIGNED_FIXED_POINT_INTEGER
+					case FLOATING_POINT_NUMBER:
+					case FIXED_POINT_INTEGER:
+					case UNSIGNED_FIXED_POINT_INTEGER:
 						ITxAttribute v = new TxAttribute(attrType, String.valueOf(cborDecoder.readFloat()));
 						ret.add(v);
 						break;
-					case 1: // ENUMERATION
-					case 5: // BIT_VECTOR
-					case 6: // LOGIC_VECTOR
-					case 12: // STRING
+					case ENUMERATION:
+					case BIT_VECTOR:
+					case LOGIC_VECTOR:
+					case STRING:
 						ITxAttribute s = new TxAttribute(attrType, strDict.get((int)cborDecoder.readInt()));
 						ret.add(s);
 						break;
@@ -281,9 +283,9 @@ public List<? extends ITxAttribute> parseAtrributes(byte[] chunk, long blockOffs
 		return ret;
 	}
 
-	private synchronized TxAttributeType getOrAddAttributeType(long tag, long type_id, String attrName) {
+	private synchronized TxAttributeType getOrAddAttributeType(long tag, DataType type, String attrName) {
 		if(!attributeTypes.containsKey(attrName)) {
-			attributeTypes.put(attrName, new TxAttributeType(attrName, DataType.values()[(int)type_id], AssociationType.values()[(int)tag-7]));
+			attributeTypes.put(attrName, new TxAttributeType(attrName, type, AssociationType.values()[(int)tag-7]));
 		} 
 		TxAttributeType attrType = attributeTypes.get(attrName);
 		return attrType;
@@ -328,7 +330,6 @@ public CborDbParser(FtrDbLoader loader, FileInputStream inputStream) {
 				long array_len = readArrayLength();
 				assert(array_len==-1);
 				CborType next = peekType();
-				int chunk_idx=0;
 				while(next != null && !break_type.isEqualType(next)) {
 					long tag = readTag();
 					switch((int)tag) {
@@ -404,7 +405,6 @@ public CborDbParser(FtrDbLoader loader, FileInputStream inputStream) {
 					}
 					}
 					next = peekType();
-					chunk_idx++;
 				}
 			} catch(IOException e) {
 				long pos = 0;
@@ -472,7 +472,6 @@ private void parseTx(TxStream txStream, long blockId, byte[] chunk) throws IOExc
 				long tx_size = cborDecoder.readArrayLength();
 				long txId = 0;
 				long genId = 0;
-				long attr_idx=0;
 				for(long i = 0; i<tx_size; ++i) {
 					long tag = cborDecoder.readTag();
 					switch((int)tag) {
@@ -501,22 +500,23 @@ private void parseTx(TxStream txStream, long blockId, byte[] chunk) throws IOExc
 					default:  { // skip over 7:begin attr, 8:record attr, 9:end attr
 						long sz = cborDecoder.readArrayLength();
 						assert(sz==3);
-						long name_id = cborDecoder.readInt();
-						String name = loader.strDict.get((int)name_id);
+						cborDecoder.readInt();
 						long type_id = cborDecoder.readInt();
-						switch((int)type_id) {
-						case 0: // BOOLEAN
+						switch(DataType.values()[(int)type_id]) {
+						case BOOLEAN:
 							cborDecoder.readBoolean();
 							break;
-						case 4: // FLOATING_POINT_NUMBER
-						case 7: // FIXED_POINT_INTEGER
-						case 8: // UNSIGNED_FIXED_POINT_INTEGER
+						case FLOATING_POINT_NUMBER: // FLOATING_POINT_NUMBER
+						case FIXED_POINT_INTEGER: // FIXED_POINT_INTEGER
+						case UNSIGNED_FIXED_POINT_INTEGER: // UNSIGNED_FIXED_POINT_INTEGER
 							cborDecoder.readFloat();
 							break;
+						case NONE: // UNSIGNED_FIXED_POINT_INTEGER
+							LOG.warn("Unsupported data type: "+type_id);
+							break;
 						default:
 							cborDecoder.readInt();
 						}
-						attr_idx++;
 					}
 					}
 				}

plugins/com.minres.scviewer.database.sqlite/pom.xml

@@ -4,7 +4,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.database.sqlite/src/com/minres/scviewer/database/sqlite/TxAttribute.java

@@ -32,7 +32,18 @@ public String getName() {
 
 	@Override
 	public DataType getDataType() {
-		return DataType.values()[scvAttribute.getData_type()];
+		int dt = scvAttribute.getData_type();
+		switch(dt) {
+		case 12:
+			return DataType.STRING;
+		case 10:
+			return DataType.POINTER;
+		default: 
+			if(dt<9) 
+				return DataType.values()[dt]; 
+			else
+				return DataType.NONE;
+		}
 	}
 
 	@Override

plugins/com.minres.scviewer.database.text/pom.xml

@@ -6,7 +6,7 @@
 	<parent>
 		<groupId>com.minres.scviewer</groupId>
 		<artifactId>com.minres.scviewer.parent</artifactId>
-		<version>2.17.2</version>
+		<version>2.17.3</version>
 		<relativePath>../..</relativePath>
 	</parent>
 	<packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.database.ui.swt/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <version>4.0.0-SNAPSHOT</version>

plugins/com.minres.scviewer.database.ui.swt/src/com/minres/scviewer/database/ui/swt/internal/RulerPainter.java

@@ -56,8 +56,9 @@ public void paintArea(Projection proj, Rectangle area) {
         int baselineY=waveCanvas.rulerHeight - 1;
         int bottom=waveCanvas.rulerHeight - 2;
 
+        long safe_scale_factor = scaleFactor!=0?scaleFactor:1;
         long startTickTime = startTime+rulerTickMinor-(startTime % rulerTickMinor);
-        long majorTickDist = rulerTickMajor/scaleFactor;
+        long majorTickDist = rulerTickMajor/safe_scale_factor;
 
         gc.setBackground(waveCanvas.getDisplay().getSystemColor(SWT.COLOR_WIDGET_BACKGROUND));
         gc.fillRectangle(new Rectangle(area.x, area.y, area.width, waveCanvas.rulerHeight));
@@ -75,7 +76,7 @@ public void paintArea(Projection proj, Rectangle area) {
         boolean drawEvery = majorTickDist>maxTextLength;
         boolean drawText=true;
         for (long tickTime = startTickTime; tickTime < endTime; tickTime+= rulerTickMinor) {
-            int x0Pos = (int) (tickTime/scaleFactor) + proj.getTranslation().x;
+            int x0Pos = (int) (tickTime/safe_scale_factor) + proj.getTranslation().x;
             if ((tickTime % rulerTickMajor) == 0) {
             	if(drawEvery ||  drawText)
             		gc.drawText(waveCanvas.timeToString(tickTime), x0Pos, area.y+textY);

plugins/com.minres.scviewer.database.vcd/pom.xml

@@ -5,7 +5,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.database/pom.xml

@@ -4,7 +4,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.database/src/com/minres/scviewer/database/DataType.java

@@ -14,46 +14,30 @@
  * The Enum DataType.
  */
 public enum DataType {
-
 	/** The boolean. */
-	BOOLEAN,
+	BOOLEAN,                      // bool
 	/** The enumeration. */
-	// bool
-	ENUMERATION,
+	ENUMERATION,                  // enum
 	/** The integer. */
-	// enum
-	INTEGER,
+	INTEGER,                      // char, short, int, long, long long, sc_int, sc_bigint
 	/** The unsigned. */
-	// char, short, int, long, long long, sc_int, sc_bigint
-	UNSIGNED, // unsigned { char, short, int, long, long long }, sc_uint,
+	UNSIGNED,                     // unsigned { char, short, int, long, long long }, sc_uint, sc_biguint
 	/** The floating point number. */
-	// sc_biguint
-	FLOATING_POINT_NUMBER,
+	FLOATING_POINT_NUMBER,        // float, double
 	/** The bit vector. */
-	// float, double
-	BIT_VECTOR,
+	BIT_VECTOR,                   // sc_bit, sc_bv
 	/** The logic vector. */
-	// sc_bit, sc_bv
-	LOGIC_VECTOR,
+	LOGIC_VECTOR,                 // sc_logic, sc_lv
 	/** The fixed point integer. */
-	// sc_logic, sc_lv
-	FIXED_POINT_INTEGER,
+	FIXED_POINT_INTEGER,          // sc_fixed
 	/** The unsigned fixed point integer. */
-	// sc_fixed
-	UNSIGNED_FIXED_POINT_INTEGER,
-	/** The record. */
-	// sc_ufixed
-	RECORD,
+	UNSIGNED_FIXED_POINT_INTEGER, // sc_ufixed
 	/** The pointer. */
-	// struct/class
-	POINTER,
-	/** The array. */
-	// T*
-	ARRAY,
+	POINTER,                      // T*
 	/** The string. */
-	// string, std::string
-	STRING, 
+	STRING,                       // string, std::string
 	/** The time. */
-	// sc_time
-	TIME
+	TIME,                         // sc_time
+	/** The void type. */
+	NONE
 }

plugins/com.minres.scviewer.e4.application.help/META-INF/MANIFEST.MF

@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: SCViewer Help
 Bundle-SymbolicName: com.minres.scviewer.e4.application.help;singleton:=true
-Bundle-Version: 2.17.2
+Bundle-Version: 2.17.3
 Bundle-Vendor: MINRES Technologies GmbH
 Automatic-Module-Name: com.minres.scviewer.e4.application.help
 Bundle-RequiredExecutionEnvironment: JavaSE-11

plugins/com.minres.scviewer.e4.application.help/pom.xml

@@ -6,7 +6,7 @@
 	<parent>
 		<groupId>com.minres.scviewer</groupId>
 		<artifactId>com.minres.scviewer.parent</artifactId>
-		<version>2.17.2</version>
+		<version>2.17.3</version>
 		<relativePath>../..</relativePath>
 	</parent>
 	<packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.e4.application/META-INF/MANIFEST.MF

@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %Bundle-Name
 Bundle-SymbolicName: com.minres.scviewer.e4.application;singleton:=true
-Bundle-Version: 2.17.2
+Bundle-Version: 2.17.3
 Bundle-Vendor: %Bundle-Vendor
 Require-Bundle: javax.inject;bundle-version="1.0.0",
  org.eclipse.core.runtime;bundle-version="3.11.1",

plugins/com.minres.scviewer.e4.application/pom.xml

@@ -6,7 +6,7 @@
 	<parent>
 		<groupId>com.minres.scviewer</groupId>
 		<artifactId>com.minres.scviewer.parent</artifactId>
-		<version>2.17.2</version>
+		<version>2.17.3</version>
 		<relativePath>../..</relativePath>
 	</parent>
 	<packaging>eclipse-plugin</packaging>

plugins/com.minres.scviewer.ui/pom.xml

@@ -4,7 +4,7 @@
   <parent>
   	<groupId>com.minres.scviewer</groupId>
   	<artifactId>com.minres.scviewer.parent</artifactId>
-  	<version>2.17.2</version>
+  	<version>2.17.3</version>
 	<relativePath>../..</relativePath>
   </parent>
   <packaging>eclipse-plugin</packaging>