Skip to content
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

feat: add tell() and error() functions to istream + ostream devices #747

Merged

Conversation

striezel
Copy link
Contributor

Description

Adds tell() and error() functions to istream_device and ostream_device classes.

The class file_stream_device already has such functions, and now both istream_device and ostream_device have them, too, making the device classes more consistent.

Closes #724.

References

See #724.

Tasklist

  • Ensure all CI builds pass
  • Review and approve

The class file_stream_device already has such functions, and now
both istream_device and ostream_device have them, too, making the
device classes more consistent.

Closes boostorg#724.
Copy link

codecov bot commented May 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.12%. Comparing base (322c4e2) to head (e602fc2).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #747   +/-   ##
========================================
  Coverage    82.12%   82.12%           
========================================
  Files          117      117           
  Lines         5355     5355           
========================================
  Hits          4398     4398           
  Misses         957      957           

@mloskot mloskot added cat/enhancement Improvements, but not fixes addressing identified bugs cat/feature New feature or functionality labels May 17, 2024
Copy link
Member

@mloskot mloskot left a comment

Choose a reason for hiding this comment

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

Awesome, thanks a lot!

@mloskot mloskot added this to the Boost 1.83+ milestone May 17, 2024
@mloskot mloskot merged commit 481b6e7 into boostorg:develop May 17, 2024
20 checks passed
@striezel striezel deleted the add_stream_device_functions branch May 17, 2024 14:30
@striezel striezel mentioned this pull request Jun 30, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/enhancement Improvements, but not fixes addressing identified bugs cat/feature New feature or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stream device "tell" method absent!
2 participants