Changeset 54 in code for trunk/bridge.go

Timestamp:

Feb 17, 2020, 11:27:48 AM (5 years ago)

Author:

contact

Message:

Don't write to downstreamConn.messages directly

Use a helper function instead. This will allow us to change
downstreamConn implementation details without having to update the whole
codebase.

File:

• trunk/bridge.go (modified) (2 diffs)

trunk/bridge.go

@@ -10 +10 @@
         }
 
-        dc.messages <- &irc.Message{
+        dc.SendMessage(&irc.Message{
                 Prefix:  dc.prefix(),
                 Command: "JOIN",
                 Params:  []string{ch.Name},
-        }
+        })
 
         if ch.Topic != "" {
-                dc.messages <- &irc.Message{
+                dc.SendMessage(&irc.Message{
                         Prefix:  dc.srv.prefix(),
                         Command: irc.RPL_TOPIC,
                         Params:  []string{dc.nick, ch.Name, ch.Topic},
-                }
+                })
         } else {
-                dc.messages <- &irc.Message{
+                dc.SendMessage(&irc.Message{
                         Prefix:  dc.srv.prefix(),
                         Command: irc.RPL_NOTOPIC,
                         Params:  []string{dc.nick, ch.Name, "No topic is set"},
-                }
+                })
         }
 
@@ -39 +39 @@
                 }
 
-                dc.messages <- &irc.Message{
+                dc.SendMessage(&irc.Message{
                         Prefix:  dc.srv.prefix(),
                         Command: irc.RPL_NAMREPLY,
                         Params:  []string{dc.nick, string(ch.Status), ch.Name, s},
-                }
+                })
         }
 
-        dc.messages <- &irc.Message{
+        dc.SendMessage(&irc.Message{
                 Prefix:  dc.srv.prefix(),
                 Command: irc.RPL_ENDOFNAMES,
                 Params:  []string{dc.nick, ch.Name, "End of /NAMES list"},
-        }
+        })
 }