We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I suspect the tally:SourceState.Connected state is set incorrectly, and will always be true: https://github.com/qmsk/e2/blob/v0.4.1/tally/state.go#L131
tally:SourceState.Connected
true
I assume that the tally:Source.xmlClient will never be nil in the current implementation.
tally:Source.xmlClient
The same applies to the Source.updateState: https://github.com/qmsk/e2/blob/master/tally/source.go#L87
Source.updateState
These should either use the closed, err state set by tally:Source.run, or that func should also set xmlClient to nil.
closed
err
tally:Source.run
xmlClient
nil
The text was updated successfully, but these errors were encountered:
Fix issues qmsk#16 & qmsk#17
733c263
We now use the source.closed & source.err states to determine source.Connected
854c888
No branches or pull requests
I suspect the
tally:SourceState.Connected
state is set incorrectly, and will always betrue
: https://github.com/qmsk/e2/blob/v0.4.1/tally/state.go#L131I assume that the
tally:Source.xmlClient
will never be nil in the current implementation.The same applies to the
Source.updateState
: https://github.com/qmsk/e2/blob/master/tally/source.go#L87These should either use the
closed
,err
state set bytally:Source.run
, or that func should also setxmlClient
tonil
.The text was updated successfully, but these errors were encountered: