Skip to content

Commit

Permalink
NetworkIdentity AddObserver message removed again because the bug was…
Browse files Browse the repository at this point in the history
… fixed a while ago and it generates too many false positives
  • Loading branch information
miwarnec committed Sep 29, 2018
1 parent 38a4ca2 commit 3063273
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions Mirror/Runtime/NetworkIdentity.cs
Expand Up @@ -818,9 +818,7 @@ internal void AddObserver(NetworkConnection conn)
if (m_ObserverConnections.Contains(conn.connectionId))
{
// if we try to add a connectionId that was already added, then
// we may have generated one that was already in use. this is
// not good and there should be a notification.
if (LogFilter.logWarn) { Debug.Log("AddObserver: connectionId=" + conn.connectionId + " already added for " + name + ". Maybe this function was called twice. Or maybe the transport layer generated the same connectionId twice, which would be a huge problem."); }
// we may have generated one that was already in use.
return;
}

Expand Down

0 comments on commit 3063273

Please sign in to comment.