-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MISC] rename view:: to views:: #1251
Conversation
f014963
to
ac3cacb
Compare
Codecov Report
@@ Coverage Diff @@
## master #1251 +/- ##
==========================================
+ Coverage 97.31% 97.36% +0.04%
==========================================
Files 221 221
Lines 8874 8912 +38
==========================================
+ Hits 8636 8677 +41
+ Misses 238 235 -3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took the initiative and tried to spot possible alignment errors, or include issues, e.g. includes that are not used by snippets etc. That is of course not part of this PR, but otherwise it will be hard to track down these things. So better fix them right now.
@@ -26,7 +26,7 @@ TYPED_TEST_CASE_P(bi_fm_index_cursor_collection_test); | |||
TYPED_TEST_P(bi_fm_index_cursor_collection_test, begin) | |||
{ | |||
std::vector<std::vector<dna4>> text{"AACGATCGGA"_dna4, "AACGATCGGA"_dna4}; | |||
auto rev_text = text | view::deep{std::view::reverse} | view::deep{view::persist}; | |||
auto rev_text = text | views::deep{std::views::reverse} | views::deep{views::persist}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
potentially missing includes for deep and persist
@@ -37,7 +37,7 @@ inline void test_search_hamming(auto it, text_t const & text, auto const & searc | |||
using char_t = typename text_t::value_type; | |||
|
|||
uint64_t const pos = std::rand() % (text.size() - query_length + 1); | |||
text_t const orig_query = text | view::slice(pos, pos + query_length); | |||
text_t const orig_query = text | views::slice(pos, pos + query_length); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
potentially missing includes for slice view
e46b04f
to
8c65076
Compare
Only concatenated_sequences has interesting changes