gs: Comments should end in a period

pkg/gatewayserver/io/io.go

@@ -310,7 +310,7 @@ func (c *Connection) HandleUp(up *ttnpb.UplinkMessage, frontendSync *FrontendClo
 
 	case gpsTime != nil:
 		gatewayTime := *ttnpb.StdTime(gpsTime)
-		// Bryan: up.Settings.Timestamp is tmst, check udp/translation.go v1Metadata and v2Metadata
+		// Bryan: up.Settings.Timestamp is tmst, check udp/translation.go v1Metadata and v2Metadata.
 		ct = c.scheduler.SyncWithGatewayAbsolute(up.Settings.Timestamp, receivedAt, gatewayTime)
 		log.FromContext(c.ctx).WithFields(log.Fields(
 			"timestamp", up.Settings.Timestamp,
@@ -319,11 +319,9 @@ func (c *Connection) HandleUp(up *ttnpb.UplinkMessage, frontendSync *FrontendClo
 			"gateway_time", gatewayTime,
 		)).Debug("Synchronized server and gateway absolute time")
 
-	// Bryan: gpsTime will be nil if
-	// Bryan: * tmms is not provided
-	// Bryan: * tmms is invalid (check where gpsTimeDelta is used)
+	// Bryan: gpsTime will be nil when tmms is not provided, or tmms is invalid (check where gpsTimeDelta is used).
 	case gpsTime == nil:
-		// Bryan: up.Settings.Timestamp is tmst, check udp/translation.go v1Metadata and v2Metadata
+		// Bryan: up.Settings.Timestamp is tmst, check udp/translation.go v1Metadata and v2Metadata.
 		ct = c.scheduler.Sync(up.Settings.Timestamp, receivedAt)
 		log.FromContext(c.ctx).WithFields(log.Fields(
 			"timestamp", up.Settings.Timestamp,

pkg/gatewayserver/scheduling/clock.go

@@ -72,15 +72,15 @@ func (c *RolloverClock) Sync(timestamp uint32, server time.Time) ConcentratorTim
 	c.absolute = (ConcentratorTime(rollovers<<32) + ConcentratorTime(timestamp)) * ConcentratorTime(time.Microsecond)
 	c.relative = timestamp
 	c.server = &server
-	// Bryan: when gateway gps time is not set or invalid, this sync function will clear gateway gps time
+	// Bryan: when gateway gps time is not set or invalid, this sync function will clear gateway gps time.
 	c.gateway = nil
 	c.synced = true
 	return c.absolute
 }
 
 // SyncWithGatewayAbsolute synchronizes the clock with the given concentrator timestamp, the server time and the
 // absolute gateway time that corresponds to the given timestamp.
-// Bryan: this function still syncs with tmst but caching gateway gps time
+// Bryan: this function still syncs with tmst but caching gateway gps time.
 func (c *RolloverClock) SyncWithGatewayAbsolute(timestamp uint32, server, gateway time.Time) ConcentratorTime {
 	ct := c.Sync(timestamp, server)
 	c.gateway = &gateway

pkg/gatewayserver/scheduling/scheduler.go

@@ -316,22 +316,13 @@ func (s *Scheduler) ScheduleAt(ctx context.Context, opts Options) (res Emission,
 	if !ok {
 		panic("clock is synced without server time")
 	}
-	// Bryan: opts.time is absolute time and only set for Class B and C
-	// Bryan: check gatewayserver/io ScheduleDown
+	// Bryan: opts.time is absolute time and only set for Class B and C, check gatewayserver/io ScheduleDown.
 	if opts.Time != nil {
 		var ok bool
 		starts, ok = s.clock.FromGatewayTime(*ttnpb.StdTime(opts.Time))
 		if !ok {
-			// Bryan: we can simply fail here to block all Class B and C transmission when gateway gps time is nil in clock.go
+			// Bryan: we can simply fail here to block all Class B and C transmission when gateway gps time is nil in clock.go.
 			return Emission{}, 0, errNoAbsoluteGatewayTime.New()
-			// if medianRTT == nil {
-			// 	return Emission{}, 0, errNoAbsoluteGatewayTime.New()
-			// }
-			// serverTime, ok := s.clock.FromServerTime(*ttnpb.StdTime(opts.Time))
-			// if !ok {
-			// 	return Emission{}, 0, errNoServerTime.New()
-			// }
-			// starts = serverTime - ConcentratorTime(*medianRTT/2)
 		}
 	} else {
 		starts = s.clock.FromTimestampTime(opts.Timestamp)