Build #844

Plan branch:
Verifies the integrety of the projects, as builds are executed immediately after a code change was detected. This plan provides no artifiacts (use a nightly build instead).

Build: #844 was successful Changes by Florian Schmaus <flo@geekplace.eu>

Build result summary

Details

Completed
Queue duration
< 1 second
Duration
6 minutes
Labels
None
Agent
Default Agent
Revision
cad63bc1078524bf5ddf2515fe656adceb93034f
Total tests
828
Successful since
#838 ()

Tests

Code commits

Author Commit Message Commit date
Florian Schmaus <flo@geekplace.eu> Florian Schmaus <flo@geekplace.eu> cad63bc1078524bf5ddf2515fe656adceb93034f Merge branch '4.4'
Florian Schmaus <flo@geekplace.eu> Florian Schmaus <flo@geekplace.eu> 8736e3e4345ad49e2e922e999770cdd31c10a1e5 m [core] Remove erroneous "assert !connected" in connect()
Connections like XMPPTCPConnection may still reported connected, if
they are, for example disconnected but resumable. This is already
accounted for in throwAlreadyConnectedExceptionIfAppropriate(), hence
the assert is unnecessary and leads to false negatives for
XMPPTCPConnection.

For the sake of completeness, the right condition, assuming
XMPPTCPConnection is used, for the assert would be:

        assert !connected || disconnectedButResumable;
Florian Schmaus <flo@geekplace.eu> Florian Schmaus <flo@geekplace.eu> 403890d9884fac25073413aa7d8ce5238e0f7770 m Smack 4.4.5-SNAPSHOT