Changeset 248 in code for trunk

Timestamp:

Apr 7, 2020, 7:54:24 PM (5 years ago)

Author:

contact

Message:

Make newMessageLogger take a *network instead of an *upstreamConn

There's no reason why messgeLogger needs access to the whole connection,
the network is enough.

Location:

trunk

Files:

• logger.go (modified) (3 diffs)
• upstream.go (modified) (1 diff)

trunk/logger.go

@@ -12 +12 @@
 
 type messageLogger struct {
-        conn   *upstreamConn
-        entity string
+        network *network
+        entity  string
 
         path string
@@ -19 +19 @@
 }
 
-func newMessageLogger(uc *upstreamConn, entity string) *messageLogger {
+func newMessageLogger(network *network, entity string) *messageLogger {
         return &messageLogger{
-                conn:   uc,
-                entity: entity,
+                network: network,
+                entity:  entity,
         }
 }
@@ -47 +47 @@
         // TODO: handle non-monotonic clock behaviour
         now := time.Now()
-        path := logPath(ml.conn.network, ml.entity, now)
+        path := logPath(ml.network, ml.entity, now)
         if ml.path != path {
                 if ml.file != nil {

trunk/upstream.go

@@ -1286 +1286 @@
         ml, ok := uc.messageLoggers[entity]
         if !ok {
-                ml = newMessageLogger(uc, entity)
+                ml = newMessageLogger(uc.network, entity)
                 uc.messageLoggers[entity] = ml
         }