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

doxygen style comment for decompress #123

Closed
wants to merge 2 commits into from

Conversation

garymm
Copy link
Owner

@garymm garymm commented Jan 6, 2024

doxygen style comment for decompress

Change-Id: Ie061dff47a33fdc8995770c7a5f98129b8788556

Change-Id: Ie5ff9ebaffe3eff70e12fd52b608fc90468e650a
Change-Id: Ie061dff47a33fdc8995770c7a5f98129b8788556
Copy link

codecov bot commented Jan 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d56844f) 82.22% compared to head (5fc53a7) 82.19%.

Additional details and impacted files
@@                              Coverage Diff                              @@
##           Ie5ff9ebaffe3eff70e12fd52b608fc90468e650a     #123      +/-   ##
=============================================================================
- Coverage                                      82.22%   82.19%   -0.03%     
=============================================================================
  Files                                             17       17              
  Lines                                            630      629       -1     
  Branches                                          38       37       -1     
=============================================================================
- Hits                                             518      517       -1     
+ Misses                                            96       94       -2     
- Partials                                          16       18       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

///
/// @returns The decompressed bytes, or an error.
/// @tparam N The size of the compressed bytes.
/// @tparam ByteAllocator The allocator to use for the decompressed bytes.
template <std::size_t N, class ByteAllocator = std::allocator<std::byte>>
auto decompress(
std::span<const std::byte, N> compressed, ByteAllocator alloc = {})
Copy link
Collaborator

Choose a reason for hiding this comment

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

auto decompress(span<const byte, N> compressed, span<byte, M> dest)
-> decompress_result;

struct decompress_result
{
DecompressError status;
span remaining_compressed;
span remaining_dest;
};

@oliverlee oliverlee force-pushed the Ie5ff9ebaffe3eff70e12fd52b608fc90468e650a branch from d56844f to f2be076 Compare January 6, 2024 18:40
Base automatically changed from Ie5ff9ebaffe3eff70e12fd52b608fc90468e650a to master January 6, 2024 18:46
@garymm
Copy link
Owner Author

garymm commented Jan 16, 2024

Changed the interface and updated docs in another PR.

@garymm garymm closed this Jan 16, 2024
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.

2 participants