mapport: remove dead code in DispatchMapPort

darosior · darosior · commit 40e5f26a3ff7 · 2024-10-25T15:02:07.000-04:00
Since there is now only two options in the MapPortProtoFlag enum, the
four possible combinations of current and enabled are already covered in
the four `if` branches.
diff --git a/src/mapport.cpp b/src/mapport.cpp
@@ -171,10 +171,6 @@ static void DispatchMapPort()
     if (g_mapport_enabled_protos & g_mapport_current_proto) {
         return;
     }
-
-    assert(g_mapport_thread.joinable());
-    assert(!g_mapport_interrupt);
-    g_mapport_interrupt();
 }
 
 static void MapPortProtoSetEnabled(MapPortProtoFlag proto, bool enabled)

Original file line number	Diff line number	Diff line change
`@@ -171,10 +171,6 @@ static void DispatchMapPort()`
`171`	`171`	`if (g_mapport_enabled_protos & g_mapport_current_proto) {`
`172`	`172`	`return;`
`173`	`173`	`}`
`174`		`-`
`175`		`- assert(g_mapport_thread.joinable());`
`176`		`- assert(!g_mapport_interrupt);`
`177`		`- g_mapport_interrupt();`
`178`	`174`	`}`
`179`	`175`
`180`	`176`	`static void MapPortProtoSetEnabled(MapPortProtoFlag proto, bool enabled)`