diff options
Diffstat (limited to 'internal/timeline')
-rw-r--r-- | internal/timeline/get.go | 5 | ||||
-rw-r--r-- | internal/timeline/index.go | 5 | ||||
-rw-r--r-- | internal/timeline/indexeditems.go | 2 | ||||
-rw-r--r-- | internal/timeline/manager.go | 3 | ||||
-rw-r--r-- | internal/timeline/prepare.go | 4 | ||||
-rw-r--r-- | internal/timeline/prune.go | 2 | ||||
-rw-r--r-- | internal/timeline/remove.go | 4 | ||||
-rw-r--r-- | internal/timeline/unprepare.go | 2 |
8 files changed, 12 insertions, 15 deletions
diff --git a/internal/timeline/get.go b/internal/timeline/get.go index bc238c276..93c869e73 100644 --- a/internal/timeline/get.go +++ b/internal/timeline/get.go @@ -222,7 +222,7 @@ func (t *timeline) getXBetweenIDs(ctx context.Context, amount int, behindID stri // a point where the items are out of the range // we're interested in. rangeF = func(e *list.Element) (bool, error) { - entry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.itemID >= behindID { // ID of this item is too high, @@ -276,7 +276,6 @@ func (t *timeline) getXBetweenIDs(ctx context.Context, amount int, behindID stri // Move the mark back one place each loop. beforeIDMark = e - //nolint:forcetypeassert if entry := e.Value.(*indexedItemsEntry); entry.itemID <= beforeID { // We've gone as far as we can through // the list and reached entries that are @@ -319,7 +318,7 @@ func (t *timeline) getXBetweenIDs(ctx context.Context, amount int, behindID stri // To preserve ordering, we need to reverse the slice // when we're finished. for e := beforeIDMark; e != nil; e = e.Prev() { - entry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.itemID == beforeID { // Don't include the beforeID diff --git a/internal/timeline/index.go b/internal/timeline/index.go index 993f7dc5d..6abb6d28d 100644 --- a/internal/timeline/index.go +++ b/internal/timeline/index.go @@ -65,7 +65,7 @@ func (t *timeline) indexXBetweenIDs(ctx context.Context, amount int, behindID st ) for e := t.items.data.Front(); e != nil; e = e.Next() { - entry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) position++ @@ -174,7 +174,6 @@ func (t *timeline) grab(ctx context.Context, amount int, behindID string, before // Don't grab more than we need to. amount-grabbed, ) - if err != nil { // Grab function already checks for // db.ErrNoEntries, so if an error @@ -280,5 +279,5 @@ func (t *timeline) OldestIndexedItemID() string { return "" } - return e.Value.(*indexedItemsEntry).itemID //nolint:forcetypeassert + return e.Value.(*indexedItemsEntry).itemID } diff --git a/internal/timeline/indexeditems.go b/internal/timeline/indexeditems.go index c2c7a19b6..9b75e7256 100644 --- a/internal/timeline/indexeditems.go +++ b/internal/timeline/indexeditems.go @@ -65,7 +65,7 @@ func (i *indexedItems) insertIndexed(ctx context.Context, newEntry *indexedItems for e := i.data.Front(); e != nil; e = e.Next() { currentPosition++ - currentEntry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + currentEntry := e.Value.(*indexedItemsEntry) // Check if we need to skip inserting this item based on // the current item. diff --git a/internal/timeline/manager.go b/internal/timeline/manager.go index df0323cdb..b4f075138 100644 --- a/internal/timeline/manager.go +++ b/internal/timeline/manager.go @@ -219,7 +219,6 @@ func (m *manager) UnprepareItemFromAllTimelines(ctx context.Context, itemID stri // Work through all timelines held by this // manager, and call Unprepare for each. m.timelines.Range(func(_ any, v any) bool { - // nolint:forcetypeassert if err := v.(Timeline).Unprepare(ctx, itemID); err != nil { errs.Append(err) } @@ -248,7 +247,7 @@ func (m *manager) getOrCreateTimeline(ctx context.Context, timelineID string) Ti i, ok := m.timelines.Load(timelineID) if ok { // Timeline already existed in sync.Map. - return i.(Timeline) //nolint:forcetypeassert + return i.(Timeline) } // Timeline did not yet exist in sync.Map. diff --git a/internal/timeline/prepare.go b/internal/timeline/prepare.go index 3344a0e73..07bde79fa 100644 --- a/internal/timeline/prepare.go +++ b/internal/timeline/prepare.go @@ -63,7 +63,7 @@ func (t *timeline) prepareXBetweenIDs(ctx context.Context, amount int, behindID if frontToBack { // Paging forwards / down. for e := t.items.data.Front(); e != nil; e = e.Next() { - entry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.itemID > behindID { l.Trace("item is too new, continuing") @@ -91,7 +91,7 @@ func (t *timeline) prepareXBetweenIDs(ctx context.Context, amount int, behindID } else { // Paging backwards / up. for e := t.items.data.Back(); e != nil; e = e.Prev() { - entry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.itemID < beforeID { l.Trace("item is too old, continuing") diff --git a/internal/timeline/prune.go b/internal/timeline/prune.go index a3a5bf9cb..5c7476956 100644 --- a/internal/timeline/prune.go +++ b/internal/timeline/prune.go @@ -63,7 +63,7 @@ func (t *timeline) Prune(desiredPreparedItemsLength int, desiredIndexedItemsLeng continue } - entry := e.Value.(*indexedItemsEntry) //nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.prepared == nil { // It's already unprepared (mood). continue diff --git a/internal/timeline/remove.go b/internal/timeline/remove.go index 693c9f9b9..86352b9fa 100644 --- a/internal/timeline/remove.go +++ b/internal/timeline/remove.go @@ -42,7 +42,7 @@ func (t *timeline) Remove(ctx context.Context, statusID string) (int, error) { var toRemove []*list.Element for e := t.items.data.Front(); e != nil; e = e.Next() { - entry := e.Value.(*indexedItemsEntry) // nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.itemID != statusID { // Not relevant. @@ -78,7 +78,7 @@ func (t *timeline) RemoveAllByOrBoosting(ctx context.Context, accountID string) var toRemove []*list.Element for e := t.items.data.Front(); e != nil; e = e.Next() { - entry := e.Value.(*indexedItemsEntry) // nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.accountID != accountID && entry.boostOfAccountID != accountID { // Not relevant. diff --git a/internal/timeline/unprepare.go b/internal/timeline/unprepare.go index 827b274d8..67a990287 100644 --- a/internal/timeline/unprepare.go +++ b/internal/timeline/unprepare.go @@ -31,7 +31,7 @@ func (t *timeline) Unprepare(ctx context.Context, itemID string) error { } for e := t.items.data.Front(); e != nil; e = e.Next() { - entry := e.Value.(*indexedItemsEntry) // nolint:forcetypeassert + entry := e.Value.(*indexedItemsEntry) if entry.itemID != itemID && entry.boostOfID != itemID { // Not relevant. |