fix up SLIP decoding and data-less qlab replies

qController/Classes/Communication/QParser.cs

@@ -81,22 +81,42 @@ namespace qController.Communication
         public void ParseCueUpdateInfo(OscMessage msg)
         {
             JToken cueUpdate = OSC2JSON(msg);
+            if(cueUpdate != null)
+            {
                 QCue cue = JsonConvert.DeserializeObject<QCue>(cueUpdate.ToString());
                 OnCueInfoUpdated(cue);
+            }else
+            {
+                Log.Debug("QPARSER - cue update info does not have any data");
+            }
         }
 
         public void ParseSelectedCueInfo(OscMessage msg){
             JToken selectedCue = OSC2JSON(msg);
+            if (selectedCue != null)
+            {
                 QCue cue = JsonConvert.DeserializeObject<QCue>(selectedCue.ToString());
                 OnSelectedCueUpdated(cue);
             }
+            else
+            {
+                Log.Debug("QPARSER - selected does not have any data");
+            }
+        }
 
         public void ParseQInfo(OscMessage msg)
         {
             JToken qInfo = OSC2JSON(msg);
+            if (qInfo != null)
+            {
                 List<QWorkspaceInfo> qWorkspaceInfo = JsonConvert.DeserializeObject<List<QWorkspaceInfo>>(qInfo.ToString());
                 OnWorkspaceInfoReceived(qWorkspaceInfo);
             }
+            else
+            {
+                Log.Debug("QPARSER - selected does not have any data");
+            }
+        }
 
         public void ParseWorkspaceInfo(OscMessage msg)
         {
@@ -131,8 +151,15 @@ namespace qController.Communication
             {
                 string cue_id = msg.Address.Split('/')[3];
                 JToken children_obj = OSC2JSON(msg);
+                if(children_obj != null)
+                {
                     List<QCue> children = JsonConvert.DeserializeObject<List<QCue>>(children_obj.ToString());
                     OnChildrenUpdated(cue_id, children);
+                }else
+                {
+                    Log.Debug("QPARSER - children info message does not have any data");
+
+                }
             }
         }
 
@@ -157,6 +184,7 @@ namespace qController.Communication
 
         protected virtual void OnPlaybackPositionUpdated(string id)
         {
+            Log.Error(id);
             PlaybackPositionUpdated?.Invoke(this, new PlaybackPositionArgs() { PlaybackPosition = id });
         }
         protected virtual void OnConnectionStatusChanged(string status, string workspace_id)

qController/Classes/Communication/SharpOSC/OscMessage.cs

@@ -17,6 +17,11 @@ namespace SharpOSC
 			Arguments.AddRange(args);
 		}
 
+        public override string ToString()
+        {
+            return "Address: " + Address + " Args: " + string.Join(",", Arguments);
+        }
+
         public override byte[] GetBytes()
 		{
 			List<byte[]> parts = new List<byte[]>();

qController/Classes/Communication/SharpOSC/OscPacket.cs

@@ -15,6 +15,11 @@ namespace SharpOSC
 				return parseMessage(OscData);
 		}
 
+		public static OscMessage GetMessage(byte[] OscData)
+		{
+			return parseMessage(OscData);
+		}
+
 		public abstract byte[] GetBytes();
 
 		#region Parse OSC packages

qController/Classes/Communication/SharpOSC/SlipFrame.cs

@@ -8,12 +8,12 @@ namespace SharpOSC
         public static readonly byte ESC = 0xdb;
         public static readonly byte ESC_END = 0xDC;
         public static readonly byte ESC_ESC = 0xDD;
+        private static List<byte> buffer = new List<byte>();
 
         public static List<byte[]> Decode(byte[] data)
         {
             List<byte[]> messages = new List<byte[]>();
 
-            List<byte> buffer = new List<byte>();
             bool escapeNext = false;
             for (int i = 0; i < data.Length; i++)
             {

qController/Classes/Communication/SharpOSC/TCPClient.cs

@@ -36,8 +36,6 @@ namespace SharpOSC
 
         TcpSharpSocketClient? tcpClient;
 
-        private List<byte> frameStream = new List<byte>();
-
         public TCPClient(string address, int port)
         {
             _port = port;
@@ -78,39 +76,28 @@ namespace SharpOSC
 
         private void DataReceived(object sender, OnClientDataReceivedEventArgs e)
         {
-            frameStream.AddRange(e.Data);
+            List<byte[]> messages = SlipFrame.Decode(e.Data);
-
+            _log.Debug("tcpclient: " +  messages.Count + " messages pulled from slip");
-            int i = 0;
-            int frameEnd = frameStream.FindIndex(1, frameByte => frameByte.Equals(SlipFrame.END));
-            while (frameEnd > 0)
-            {
-                List<byte> frame = frameStream.GetRange(0, frameEnd + 1);
-                frameStream.RemoveRange(0, frameEnd + 1);
-                List<byte[]> messages = SlipFrame.Decode(frame.ToArray());
             foreach (var message in messages)
             {
                 try
                 {
-                        OscPacket packet = OscPacket.GetPacket(message);
-                        OscMessage responseMessage = (OscMessage)packet;
-                        if (packet == null)
-                        {
-                            _log.Error("packet is null");
-                        }
 
+                    OscMessage responseMessage = OscPacket.GetMessage(message);
                     if (responseMessage == null)
                     {
-                            _log.Error("responeMessage is null");
+                        _log.Error("responseMessage is null");
-                        }
+                    } else
+                    {
                         OnMessageReceived(responseMessage);
                     }
+
+                }
                 catch (Exception ex)
                 {
                     _log.Error($"Exception parsing OSC message: {ex.ToString()}");
                 }
             }
-                frameEnd = frameStream.FindIndex(0, frameByte => frameByte.Equals(SlipFrame.END));
-            }
         }
 
         private void ClientConnected(object sender, OnClientConnectedEventArgs e)

qController/Pages/ControlPage.xaml.cs

@@ -289,6 +289,9 @@ namespace qController
                     }
                     qCell.UpdateSelectedCue(cue);
                 });
+            } else
+            {
+                Log.Error("Playback Position has been updated but cue is null?");
             }
         }