[Fixed #17] Offline issues #18
Conversation
IOException must be thrown for the outer logic to work correct.
| @@ -219,11 +229,11 @@ public void run(){ | |||
| // Send data, reconnect if needed | |||
| while (true){ | |||
| try{ | |||
| stream.write( msg); | |||
| stream.flush(); | |||
| stream.write( msg); | |||
There was a problem hiding this comment.
Any reason for removing the indentation?
|
Looks good, not sure why the indentation was moved though. |
|
Hi Stephen! Sorry about that, I noticed that you use tabs instead of spaces when I I'll update the commit. Best Regards, 2015-03-31 11:54 GMT+02:00 Stephen Hynes notifications@github.com:
|
|
Hi again, There is a new commit with the indentation fixes. /Jim 2015-03-31 13:07 GMT+02:00 Jim Arnell jim@arnellconsulting.com:
|
|
Great thanks! Ill merge this in and update the jar |
[Fixed #17] Offline issues
IOException must be thrown for the outer logic to work correctly.
Otherwise, the stream will be null which leads to NPE that exits the publishing loop.
Online -> Offline -> Online works fine now