In https://github.com/Flowdalic/Smack/commit/5a92792489249f5b4788f513fb0b4cf0010e16 8c#commitcomment-7057418 you suggest to have a tri-state variable for "success" / "failure" / "timeout". I think it would be better to have a (atomic)boolean, and throw instead, as that makes the checking logic in the code more consistent.
↧