Skip to content

Breaking: Remove file::rdbuf#173

Merged
bugdea1er merged 1 commit intomainfrom
rdbuf
Feb 15, 2025
Merged

Breaking: Remove file::rdbuf#173
bugdea1er merged 1 commit intomainfrom
rdbuf

Conversation

@bugdea1er
Copy link
Copy Markdown
Owner

file (which extends std::iostream) still provides rdbuf() -> std::streambuf which can be used for lower level I/O

std::filebuf (and tmp::filebuf) only provides open, is_open and close methods:

  • open - we don't want the user to be able to reopen another file from the temporary file
  • close - we don't want file to exist in a closed state
  • is_open - will only return false for relocated file objects

@bugdea1er bugdea1er changed the title Remove file::rdbuf Breaking: Remove file::rdbuf Feb 14, 2025
@bugdea1er bugdea1er merged commit 80f0d49 into main Feb 15, 2025
15 checks passed
@bugdea1er bugdea1er deleted the rdbuf branch February 15, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant