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

fix missing include header file#343

Merged
kkajrewicz merged 1 commit intopmem:stable-1.6from
szyrom:fix_missing_header
Jun 26, 2019
Merged

fix missing include header file#343
kkajrewicz merged 1 commit intopmem:stable-1.6from
szyrom:fix_missing_header

Conversation

@szyrom
Copy link
Copy Markdown

@szyrom szyrom commented Jun 25, 2019

stdexcept is needed in slice.hpp for std::out_of_range support


This change is Reviewable

@pmem-bot
Copy link
Copy Markdown
Contributor

@kkajrewicz and @kilobyte please review this pull request

stdexcept is needed in slice.hpp for std::out_of_range support
@szyrom szyrom force-pushed the fix_missing_header branch from d59ca07 to 93784f5 Compare June 25, 2019 16:28
@szyrom szyrom changed the base branch from master to stable-1.6 June 25, 2019 16:29
Copy link
Copy Markdown
Contributor

@igchor igchor 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: 0 of 1 files reviewed, all discussions resolved

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 25, 2019

Codecov Report

Merging #343 into stable-1.6 will increase coverage by 0.08%.
The diff coverage is 96.15%.

Impacted file tree graph

@@              Coverage Diff               @@
##           stable-1.6     #343      +/-   ##
==============================================
+ Coverage       95.94%   96.03%   +0.08%     
==============================================
  Files              32       32              
  Lines            3426     3175     -251     
==============================================
- Hits             3287     3049     -238     
+ Misses            139      126      -13
Flag Coverage Δ
#tests_clang_debug_cpp17 96.01% <97.22%> (+0.19%) ⬆️
#tests_gcc_debug 96.21% <96.15%> (-0.36%) ⬇️
#tests_gcc_release_cpp17_no_valgrind 95.3% <95.45%> (-0.41%) ⬇️
Impacted Files Coverage Δ
include/libpmemobj++/experimental/basic_string.hpp 99.82% <ø> (-0.06%) ⬇️
include/libpmemobj++/experimental/vector.hpp 98.85% <ø> (ø) ⬆️
include/libpmemobj++/experimental/slice.hpp 100% <ø> (ø) ⬆️
.../libpmemobj++/experimental/concurrent_hash_map.hpp 93.08% <ø> (+1.66%) ⬆️
...nclude/libpmemobj++/detail/persistent_pool_ptr.hpp 96.15% <96.15%> (ø) ⬆️

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 bc4128d...93784f5. Read the comment docs.

Copy link
Copy Markdown
Contributor

@kkajrewicz kkajrewicz 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 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@kkajrewicz kkajrewicz merged commit dd9ec04 into pmem:stable-1.6 Jun 26, 2019
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.

6 participants