Minor cleanup of other unsafe impl

src/function.rs

@@ -108,9 +108,6 @@ fn allow_autoload(&self, name: &wstr) -> bool {
     })
 });
 
-// Safety: global lock.
-unsafe impl Send for FunctionSet {}
-
 /// Make sure that if the specified function is a dynamically loaded function, it has been fully
 /// loaded. Note this executes fish script code.
 pub fn load(name: &wstr, parser: &Parser) -> bool {

src/topic_monitor.rs

@@ -336,6 +336,7 @@ pub struct TopicMonitor {
 }
 
 // safety: this is only needed for tests
+#[cfg(test)]
 unsafe impl Sync for TopicMonitor {}
 
 /// The principal topic monitor.