Clear a whole bunch of lint issues

clientapi/routing/peekroom.go

@@ -64,7 +64,6 @@ func PeekRoomByIDOrAlias(
 	// if this user is already joined to the room, we let them peek anyway
 	// (given they might be about to part the room, and it makes things less fiddly)
 
-
 	// Peeking stops if none of the devices who started peeking have been
 	// /syncing for a while, or if everyone who was peeking calls /leave
 	// (or /unpeek with a server_name param? or DELETE /peek?)

roomserver/internal/perform/perform_peek.go

@@ -20,8 +20,8 @@ import (
 	"fmt"
 	"strings"
 
-	"github.com/matrix-org/dendrite/internal/config"
 	fsAPI "github.com/matrix-org/dendrite/federationsender/api"
+	"github.com/matrix-org/dendrite/internal/config"
 	"github.com/matrix-org/dendrite/roomserver/api"
 	"github.com/matrix-org/dendrite/roomserver/internal/input"
 	"github.com/matrix-org/dendrite/roomserver/storage"
@@ -39,7 +39,6 @@ type Peeker struct {
 	Inputer *input.Inputer
 }
 
-
 // PerformPeek handles peeking into matrix rooms, including over federation by talking to the federationsender.
 func (r *Peeker) PerformPeek(
 	ctx context.Context,
@@ -164,7 +163,7 @@ func (r *Peeker) performPeekRoomByID(
 	// XXX: we should probably factor out history_visibility checks into a common utility method somewhere
 	// which handles the default value etc.
 	var worldReadable = false
-	ev, err := r.DB.GetStateEvent(ctx, roomID, "m.room.history_visibility", "")
+	ev, _ := r.DB.GetStateEvent(ctx, roomID, "m.room.history_visibility", "")
 	if ev != nil {
 		content := map[string]string{}
 		if err = json.Unmarshal(ev.Content(), &content); err != nil {
@@ -203,5 +202,5 @@ func (r *Peeker) performPeekRoomByID(
 	// it will have been overwritten with a room ID by performPeekRoomByAlias.
 	// We should now include this in the response so that the CS API can
 	// return the right room ID.
-	return roomID, nil;
+	return roomID, nil
 }

syncapi/storage/shared/syncserver.go

@@ -987,6 +987,7 @@ func (d *Database) fetchMissingStateEvents(
 // exclusive of oldPos, inclusive of newPos, for the rooms in which
 // the user has new membership events.
 // A list of joined room IDs is also returned in case the caller needs it.
+// nolint:gocyclo
 func (d *Database) getStateDeltas(
 	ctx context.Context, device *userapi.Device, txn *sql.Tx,
 	r types.Range, userID string,

syncapi/storage/sqlite3/peeks_table.go

@@ -178,7 +178,7 @@ func (s *peekStatements) SelectPeekingDevices(
 			return nil, err
 		}
 		devices := result[roomID]
-		devices = append(devices, types.PeekingDevice{userID, deviceID})
+		devices = append(devices, types.PeekingDevice{UserID: userID, DeviceID: deviceID})
 		result[roomID] = devices
 	}
 	return result, nil

syncapi/sync/notifier.go

@@ -246,7 +246,6 @@ func (n *Notifier) wakeupUsers(userIDs []string, peekingDevices []types.PeekingD
 		}
 	}
 
-	if peekingDevices != nil {
 	for _, peekingDevice := range peekingDevices {
 		// TODO: don't bother waking up for devices whose users we already woke up
 		if stream := n.fetchUserDeviceStream(peekingDevice.UserID, peekingDevice.DeviceID, false); stream != nil {
@@ -254,7 +253,6 @@ func (n *Notifier) wakeupUsers(userIDs []string, peekingDevices []types.PeekingD
 		}
 	}
 }
-}
 
 // wakeupUserDevice will wake up the sync stream for a specific user device. Other
 // device streams will be left alone.
@@ -332,13 +330,12 @@ func (n *Notifier) joinedUsers(roomID string) (userIDs []string) {
 	return n.roomIDToJoinedUsers[roomID].values()
 }
 
-
 // Not thread-safe: must be called on the OnNewEvent goroutine only
 func (n *Notifier) addPeekingDevice(roomID, userID, deviceID string) {
 	if _, ok := n.roomIDToPeekingDevices[roomID]; !ok {
 		n.roomIDToPeekingDevices[roomID] = make(peekingDeviceSet)
 	}
-	n.roomIDToPeekingDevices[roomID].add(types.PeekingDevice{userID, deviceID})
+	n.roomIDToPeekingDevices[roomID].add(types.PeekingDevice{UserID: userID, DeviceID: deviceID})
 }
 
 // Not thread-safe: must be called on the OnNewEvent goroutine only
@@ -347,7 +344,7 @@ func (n *Notifier) removePeekingDevice(roomID, userID, deviceID string) {
 		n.roomIDToPeekingDevices[roomID] = make(peekingDeviceSet)
 	}
 	// XXX: is this going to work as a key?
-	n.roomIDToPeekingDevices[roomID].remove(types.PeekingDevice{userID, deviceID})
+	n.roomIDToPeekingDevices[roomID].remove(types.PeekingDevice{UserID: userID, DeviceID: deviceID})
 }
 
 // Not thread-safe: must be called on the OnNewEvent goroutine only
@@ -358,8 +355,6 @@ func (n *Notifier) PeekingDevices(roomID string) (peekingDevices []types.Peeking
 	return n.roomIDToPeekingDevices[roomID].values()
 }
 
-
-
 // removeEmptyUserStreams iterates through the user stream map and removes any
 // that have been empty for a certain amount of time. This is a crude way of
 // ensuring that the userStreams map doesn't grow forver.