Skip to content

S
swiftshader
Commit 6193cb18 by Ben Clayton
Options

tests/regres/export_to_sheets: Fix off-by-one in loop

We weren't looking at the most recent change. Kinda important when that's typically the new test list you're trying to export. Change-Id: I0e312dbde0ae54feb0641a658eebc73bbdb47eaa Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26928Reviewed-by: 's avatarNicolas Capens <nicolascapens@google.com> Tested-by: 's avatarBen Clayton <bclayton@google.com>
parent abb57857
Showing with 1 addition and 1 deletion
tests/regres/export_to_sheets.go
...@@ -124,7 +124,7 @@ func run() error { ...@@ -124,7 +124,7 @@ func run() error {
rowInsertionPoint := 1 + len(hashValues) rowInsertionPoint := 1 + len(hashValues)
for i := len(changes) - 1; i > 0; i-- { for i := len(changes) - 1; i >= 0; i-- {
change := changes[i] change := changes[i]
if !strings.HasPrefix(change.Subject, consts.TestListUpdateCommitSubjectPrefix) { if !strings.HasPrefix(change.Subject, consts.TestListUpdateCommitSubjectPrefix) {
continue continue
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment