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
Demote FitsException to a runtime exception #519
Conversation
…tateException" This reverts commit a46ccda.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #519 +/- ##
============================================
+ Coverage 97.65% 97.69% +0.04%
- Complexity 4779 4790 +11
============================================
Files 207 209 +2
Lines 13796 13781 -15
Branches 2122 2123 +1
============================================
- Hits 13472 13463 -9
+ Misses 122 117 -5
+ Partials 202 201 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
…rameter.nextFreeZVal()
Since the inception of this library
FitsException
was a hard exception that forced programmers to deal with it. However, its meaning is much more similar conceptually to Java'sIllegalStateException
-- which is a soft runtime exception. Therefore, let's changeFitsException
(and its derivatives such asHeaderCardExcepton
) to be a FITS-specific subclass ofIllegalStateException
going forward.The change is entirely backward compatible from a developer's point of view. The
catch
blocks that were necessary before still work the same, only they are now optional.As a result of the change the internal
nom.fits.compression.provider.param.api.IHeaderAccess
interface, its subclasses and usage have become obsoleted and unnecessary. These have been marked as deprecated to be removed in a future release (such as 2.0).Some further tweaks in PR #530