From f7f52986f3a5a9b6d6b0c48f30c7a935c7428144 Mon Sep 17 00:00:00 2001
From: Neil Alexander <neilalexander@users.noreply.github.com>
Date: Thu, 23 Jan 2020 16:03:55 +0000
Subject: [PATCH] Remove prevPDUPos as dead now that backwardTopologyPos is
 used instead

---
 syncapi/storage/postgres/syncserver.go | 19 -------------------
 1 file changed, 19 deletions(-)

diff --git a/syncapi/storage/postgres/syncserver.go b/syncapi/storage/postgres/syncserver.go
index e1e8b983f..621aec957 100644
--- a/syncapi/storage/postgres/syncserver.go
+++ b/syncapi/storage/postgres/syncserver.go
@@ -812,25 +812,6 @@ func (d *SyncServerDatasource) addRoomDeltaToResponse(
 	recentEvents := d.StreamEventsToEvents(device, recentStreamEvents)
 	delta.stateEvents = removeDuplicates(delta.stateEvents, recentEvents) // roll back
 
-	var prevPDUPos types.StreamPosition
-
-	if len(recentEvents) == 0 {
-		if len(delta.stateEvents) == 0 {
-			// Don't bother appending empty room entries
-			return nil
-		}
-
-		// If full_state=true and since is already up to date, then we'll have
-		// state events but no recent events.
-		prevPDUPos = toPos - 1
-	} else {
-		prevPDUPos = recentStreamEvents[0].StreamPosition - 1
-	}
-
-	if prevPDUPos <= 0 {
-		prevPDUPos = 1
-	}
-
 	var backwardTopologyPos types.StreamPosition
 	backwardTopologyPos, err = d.getBackwardTopologyPos(ctx, recentStreamEvents)
 	if err != nil {