F Improved state handling in connect/re-connect, added callbacks in case Connect also fails

Author: jimm98y

src/SharpRTSPClient/RTSPClient.cs

@@ -228,6 +228,9 @@ public void Connect(
             RemoteCertificateValidationCallback userCertificateSelectionCallback = null, 
             bool autoReconnect = false)
         {
+            if (_rtspClient != null)
+                throw new InvalidOperationException("You must first call Stop() before re-connecting!");
+
             _logger.LogDebug("Connecting to {url} ", uri);
 
             this._uri = uri;
@@ -268,13 +271,15 @@ public void Connect(
             {
                 _rtspSocketStatus = RtspStatus.ConnectFailed;
                 _logger.LogWarning("Error - did not connect");
+                Stopped?.Invoke(this, EventArgs.Empty);
                 return;
             }
 
             if (!_rtspSocket.Connected)
             {
                 _rtspSocketStatus = RtspStatus.ConnectFailed;
                 _logger.LogWarning("Error - did not connect");
+                Stopped?.Invoke(this, EventArgs.Empty);
                 return;
             }
 
@@ -335,7 +340,7 @@ public void Connect(
         public void TryReconnect()
         {
             if (_uri == null)
-                throw new InvalidOperationException("Reconnect can only be called after calling Connect!");
+                throw new InvalidOperationException("You must first call Connect() before re-connecting!");
 
             Connect(_uri, _rtpTransport, _credentials, _mediaRequest, _playbackSession, _userCertificateSelectionCallback, _autoReconnect);
         }
@@ -504,6 +509,8 @@ private void StopClient()
                 rtspClient.Stop();
                 _rtspClient = null;
             }
+
+            _rtspSocket = null; // closed by rtspClient.Stop()
         }
 
         /// <summary>