Add unsigned read

ChrisPulman · Oct 19, 2023 · 3733aa0 · 3733aa0
1 parent a475127
commit 3733aa0
Show file tree

Hide file tree

Showing 2 changed files with 72 additions and 1 deletion.
diff --git a/Version.json b/Version.json
@@ -1,6 +1,6 @@
 {
     "$schema": "https://raw.githubusercontent.com/dotnet/Nerdbank.GitVersioning/main/src/NerdBank.GitVersioning/version.schema.json",
-    "version": "1.2.1",
+    "version": "1.2.2",
     "publicReleaseRefSpec": [
         "^refs/heads/master$",
         "^refs/heads/main$"

diff --git a/src/S7PlcRx/RxS7.cs b/src/S7PlcRx/RxS7.cs
@@ -809,6 +809,16 @@ private void GetTagValue(Tag? tag)
                                 return Read<ushort[]>(tag, DataType.DataBlock, dB, dbIndex, VarType.Word);
                             }
 
+                            if (tag.Type == typeof(short[]))
+                            {
+                                return Read<short[]>(tag, DataType.DataBlock, dB, dbIndex, VarType.Word);
+                            }
+
+                            if (tag.Type == typeof(short))
+                            {
+                                return Read<short>(tag, DataType.DataBlock, dB, dbIndex, VarType.Word);
+                            }
+
                             return Read<ushort>(tag, DataType.DataBlock, dB, dbIndex, VarType.Word);
 
                         case "DBD":
@@ -873,6 +883,16 @@ private void GetTagValue(Tag? tag)
                         return Read<ushort[]>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
                     }
 
+                    if (tag.Type == typeof(short[]))
+                    {
+                        return Read<short[]>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
+                    }
+
+                    if (tag.Type == typeof(short))
+                    {
+                        return Read<short>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
+                    }
+
                     return Read<ushort>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
 
                 case "ED":
@@ -883,6 +903,16 @@ private void GetTagValue(Tag? tag)
                         return Read<uint[]>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
                     }
 
+                    if (tag.Type == typeof(int[]))
+                    {
+                        return Read<int[]>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
+                    }
+
+                    if (tag.Type == typeof(int))
+                    {
+                        return Read<int>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
+                    }
+
                     return Read<uint>(tag, DataType.Input, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
 
                 case "AB":
@@ -903,6 +933,16 @@ private void GetTagValue(Tag? tag)
                         return Read<ushort[]>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
                     }
 
+                    if (tag.Type == typeof(short[]))
+                    {
+                        return Read<short[]>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
+                    }
+
+                    if (tag.Type == typeof(short))
+                    {
+                        return Read<short>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
+                    }
+
                     return Read<ushort>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
 
                 case "AD":
@@ -913,6 +953,16 @@ private void GetTagValue(Tag? tag)
                         return Read<uint[]>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
                     }
 
+                    if (tag.Type == typeof(int[]))
+                    {
+                        return Read<int[]>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
+                    }
+
+                    if (tag.Type == typeof(int))
+                    {
+                        return Read<int>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
+                    }
+
                     return Read<uint>(tag, DataType.Output, 0, int.Parse(correctVariable.Substring(2)), VarType.DWord);
 
                 case "MB":
@@ -933,6 +983,16 @@ private void GetTagValue(Tag? tag)
                         return Read<ushort[]>(tag, DataType.Memory, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
                     }
 
+                    if (tag.Type == typeof(short[]))
+                    {
+                        return Read<short[]>(tag, DataType.Memory, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
+                    }
+
+                    if (tag.Type == typeof(short))
+                    {
+                        return Read<short>(tag, DataType.Memory, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
+                    }
+
                     return Read<ushort>(tag, DataType.Memory, 0, int.Parse(correctVariable.Substring(2)), VarType.Word);
 
                 case "MD":
@@ -987,6 +1047,16 @@ private void GetTagValue(Tag? tag)
                                 return Read<ushort[]>(tag, DataType.Counter, 0, int.Parse(correctVariable.Substring(2)), VarType.Counter);
                             }
 
+                            if (tag.Type == typeof(short[]))
+                            {
+                                return Read<short[]>(tag, DataType.Counter, 0, int.Parse(correctVariable.Substring(2)), VarType.Counter);
+                            }
+
+                            if (tag.Type == typeof(short))
+                            {
+                                return Read<short>(tag, DataType.Counter, 0, int.Parse(correctVariable.Substring(2)), VarType.Counter);
+                            }
+
                             return Read<ushort>(tag, DataType.Counter, 0, int.Parse(correctVariable.Substring(1)), VarType.Counter);
 
                         default:
@@ -1206,6 +1276,7 @@ private bool Write(Tag tag, DataType dataType, int db, int startByteAdr, object?
         byte[] package;
         switch (tag.Type.Name)
         {
+            // TODO: fix Bit
             case "Byte":
                 package = new byte[] { (byte)(value ?? Convert.ChangeType(tag.NewValue, typeof(byte)))! };
                 break;