event/Loop: non-recursive Break() implementation

Simply set the "quit" flag and wake up the thread. This works even if we're inside this thread. Setting "quit" to a new value without mutex protection is usually not safe, but good enough here.

event/Loop: non-recursive Break() implementation
1f119597 · Max Kellermann · 87fce8ef · 1f119597
Commit 1f119597 authored Jan 04, 2014 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

Loop.cxx src/event/Loop.cxx +2 -4

No files found.
--- a/src/event/Loop.cxx
+++ b/src/event/Loop.cxx
@@ -52,10 +52,8 @@ EventLoop::~EventLoop()
 void
 EventLoop::Break()
 {
-	if (IsInside())
-		quit = true;
-	else
-		AddCall([this]() { Break(); });
+	quit = true;
+	wake_fd.Write();
 }

 bool