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

Unified Diff: courgette/third_party/qsufsort_unittest.cc

Issue 1914303002: [Courgette] Fix sorting in QSufSort's split(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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
« no previous file with comments | « courgette/third_party/qsufsort.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: courgette/third_party/qsufsort_unittest.cc
diff --git a/courgette/third_party/qsufsort_unittest.cc b/courgette/third_party/qsufsort_unittest.cc
index 204a09cd1cfa49456cc772b58cf472ce68dd5e0f..81a2c7210c794e2ba714d2376d92a4cd9e4b6ac9 100644
--- a/courgette/third_party/qsufsort_unittest.cc
+++ b/courgette/third_party/qsufsort_unittest.cc
@@ -38,7 +38,7 @@ TEST(QSufSortTest, Sort) {
// Generate the suffix array as I.
std::vector<int> I(len + 1);
std::vector<int> V(len + 1);
- courgette::qsuf::qsufsort<int*>(&I[0], &V[0], s, len);
+ courgette::qsuf::qsufsort<std::vector<int>&>(I, V, s, len);
// Expect that I[] is a permutation of [0, len].
std::vector<int> I_sorted(I);
@@ -71,7 +71,7 @@ TEST(QSufSortTest, Search) {
reinterpret_cast<const unsigned char*>(old_str);
std::vector<int> I(old_size + 1);
std::vector<int> V(old_size + 1);
- courgette::qsuf::qsufsort<int*>(&I[0], &V[0], old_buf, old_size);
+ courgette::qsuf::qsufsort<std::vector<int>&>(I, V, old_buf, old_size);
// Test queries.
const struct {
« no previous file with comments | « courgette/third_party/qsufsort.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698