Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(300)

Unified Diff: server/logdog/storage/archive/storage.go

Issue 1672833003: LogDog: Add log rendering view. Base URL: https://github.com/luci/luci-go@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: server/logdog/storage/archive/storage.go
diff --git a/server/logdog/storage/archive/storage.go b/server/logdog/storage/archive/storage.go
index 159cb27d11b7aed236189995b982172c3f8b003e..a49cd26fad044b466ad9b592c5a9e53c97304f9f 100644
--- a/server/logdog/storage/archive/storage.go
+++ b/server/logdog/storage/archive/storage.go
@@ -320,7 +320,7 @@ func (s *storageImpl) buildGetStrategy(req *storage.GetRequest, idx *logpb.LogIn
// for sure that we have one LogEntry per index entry, so that's the best we
// can do.
if req.Limit > 0 {
- if ub := startIdx + req.Limit; ub < len(idx.Entries) {
+ if ub := int64(startIdx) + req.Limit; ub < int64(len(idx.Entries)) {
st.to = int64(idx.Entries[ub].Offset)
}
}

Powered by Google App Engine
This is Rietveld 408576698