-
Notifications
You must be signed in to change notification settings - Fork 365
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
nonPublicPartOfAPI
for EpochMessage
reported by revapi
#2259
Comments
For me this seems to be a leftover
and I would prefer to change that to |
boaks
added a commit
that referenced
this issue
Oct 10, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
Just in the case, you have some time left, you may check branch |
boaks
added a commit
that referenced
this issue
Oct 11, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
boaks
added a commit
that referenced
this issue
Oct 14, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
boaks
added a commit
that referenced
this issue
Oct 15, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
boaks
added a commit
that referenced
this issue
Oct 15, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
boaks
added a commit
that referenced
this issue
Oct 20, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
boaks
added a commit
to boaks/californium
that referenced
this issue
Oct 23, 2024
Issue: eclipse-californium#2259 Signed-off-by: Achim Kraus <[email protected]>
boaks
added a commit
that referenced
this issue
Oct 23, 2024
Issue: #2259 Signed-off-by: Achim Kraus <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When building leshan, revapi reports something wrong with californium API. (I think this is not new but before I just ignored all error from californium and I changed my build config about that recently)
Report :
nonPublicPartOfAPI
is described here.I copy/paste the
exampleUseChainInNewApi
here because this is the most important part to understand the issue :Solution ?
I think
EpochMessage
class should be protected instead of private ?I know this is part of @NoPublicAPI but even if this is not intended to be used the visibility seems inconsistent.
(tested with Cf 3.12.0)
The text was updated successfully, but these errors were encountered: