Skip to content
This repository was archived by the owner on Mar 22, 2023. It is now read-only.

Merge stable-1.6 into stable-1.7#724

Merged
szyrom merged 4 commits intostable-1.7from
stable-1.6
Apr 17, 2020
Merged

Merge stable-1.6 into stable-1.7#724
szyrom merged 4 commits intostable-1.7from
stable-1.6

Conversation

@igchor
Copy link
Copy Markdown
Contributor

@igchor igchor commented Apr 16, 2020

This change is Reviewable

igchor and others added 4 commits March 27, 2020 10:17
of unsigned.

The reason is that Windows cannot decide for a call like this:

`it[0] = 10`

whether to use:
* operator[](unsigned) -- one conversion from signed to unsigned
* operator T*() and then build in operator[](signed) -- one conversion
contigous_iterator: make operator[] take signed integral instead
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2020

Codecov Report

Merging #724 into stable-1.7 will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           stable-1.7     #724   +/-   ##
===========================================
  Coverage       96.67%   96.67%           
===========================================
  Files              32       32           
  Lines            3484     3486    +2     
===========================================
+ Hits             3368     3370    +2     
  Misses            116      116           
Flag Coverage Δ
#tests_clang_debug_cpp17 97.13% <100.00%> (-0.12%) ⬇️
#tests_gcc_debug 95.79% <100.00%> (+0.22%) ⬆️
#tests_gcc_release_cpp17_no_valgrind 86.61% <40.00%> (+0.14%) ⬆️
Impacted Files Coverage Δ
.../libpmemobj++/experimental/contiguous_iterator.hpp 100.00% <100.00%> (ø)
include/libpmemobj++/experimental/slice.hpp 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a14df5...3efec23. Read the comment docs.

Copy link
Copy Markdown
Member

@lukaszstolarczuk lukaszstolarczuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 3 of 3 files at r3.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Copy Markdown

@szyrom szyrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@szyrom szyrom merged commit 0562a0e into stable-1.7 Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants