Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Commit

Permalink
split assertion
Browse files Browse the repository at this point in the history
- so it's distinguishable in log output
  • Loading branch information
MartinNowak committed Jul 2, 2012
1 parent 8f8d4b3 commit 5b4a78c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/core/sync/condition.d
Expand Up @@ -615,7 +615,9 @@ version( unittest )
Thread.yield();
}
thread.join();
assert( waiting && alertedOne && !alertedTwo );
assert( waiting );
assert( alertedOne );
assert( !alertedTwo );
}


Expand Down

2 comments on commit 5b4a78c

@dnadlinger
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might (unlikely, but I thought I would mention it) make some race conditions disappear, though, due to the tiny overhead of calling assert.

@MartinNowak
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I thought that too when the first test passed, but apparently it doesn't.

Please sign in to comment.