Merge pull request #244 from SandTripper/fix_list_rpop_bug

qishenonly · web-flow · commit 562634021e82 · 2023-07-26T17:23:31.000+08:00
fix the list rpop func bug
diff --git a/structure/list.go b/structure/list.go
@@ -212,7 +212,7 @@ func (l *ListStructure) RPop(key string) (interface{}, error) {
 
 	// Find the new tail
 	newTail := lst.Head
-	for i := 0; i < lst.Length-1; i++ {
+	for i := 0; i < lst.Length-2; i++ {
 		newTail = newTail.Next
 	}
 	popValue := newTail.Next.Value

Original file line number	Diff line number	Diff line change
`@@ -212,7 +212,7 @@ func (l *ListStructure) RPop(key string) (interface{}, error) {`
`212`	`212`
`213`	`213`	`// Find the new tail`
`214`	`214`	`newTail := lst.Head`
`215`		`- for i := 0; i < lst.Length-1; i++ {`
	`215`	`+ for i := 0; i < lst.Length-2; i++ {`
`216`	`216`	`newTail = newTail.Next`
`217`	`217`	`}`
`218`	`218`	`popValue := newTail.Next.Value`