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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Clean some Exception about LwM2mPath validation. #1082

Closed
sbernard31 opened this issue Aug 18, 2021 · 1 comment
Closed

Clean some Exception about LwM2mPath validation. #1082

sbernard31 opened this issue Aug 18, 2021 · 1 comment
Labels
core Impact core of Leshan housekeeping Refactoring, cleaning code or API

Comments

@sbernard31
Copy link
Contributor

Currently, LwM2mPath constructor can raise LwM2mNodeException which does not make so much sense.

Maybe it should raise IllegalArgumentException instead ? 馃
@sbernard31 sbernard31 added core Impact core of Leshan housekeeping Refactoring, cleaning code or API labels Aug 18, 2021
sbernard31 added a commit that referenced this issue Dec 1, 2021
@sbernard31
#1082: Create dedicated LwM2mPathInvalidException.
0cba977
@sbernard31 sbernard31 mentioned this issue Dec 1, 2021
Merged
@sbernard31
Copy link
Contributor Author

Integrated in master (since #1169 is merged)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core Impact core of Leshan housekeeping Refactoring, cleaning code or API
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sbernard31