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

Custom exceptionsand greater type safety. #11

Merged
merged 6 commits into from
Dec 5, 2023
Merged

Custom exceptionsand greater type safety. #11

merged 6 commits into from
Dec 5, 2023

Conversation

arbCoding
Copy link
Owner

Updated to use custom exception (sacfmt::io_error).

Updated float_to_binary and double_to_binary to use a typesafe conversion (float/double -> correctly-sized uint -> bitset instead of float/double -> bitset).

noexcept specifier has been added to functions/methods that won't throw exceptions (basically everything except explicit I/O operations, which cannot be guaranteed.

…ween floats/doubles and bitset). noexcept specifier added to functions that won't throw exceptions (most things except read/write from a file stream, which cannot be guaranteed). Minor documentation updates.
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1b74ff5) 99.14% compared to head (322b6fb) 99.71%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   99.14%   99.71%   +0.57%     
==========================================
  Files           1        2       +1     
  Lines         940     1069     +129     
==========================================
+ Hits          932     1066     +134     
+ Misses          8        3       -5     

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

@arbCoding arbCoding merged commit 607c74d into main Dec 5, 2023
7 checks passed
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