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

API-alignment bugs with EmptyQuantity #108

Open
2 tasks
ckittl opened this issue May 18, 2021 · 0 comments
Open
2 tasks

API-alignment bugs with EmptyQuantity #108

ckittl opened this issue May 18, 2021 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@ckittl
Copy link
Member

ckittl commented May 18, 2021

Currently, the EmptyQuantity is not totally aligned with the concepts of the measurement API. The following issues should be addressed:

  • AbstractQuantity#toString fails rendering the null value to a sensible output
  • AbstractQuantity#compareTo(Quantity) fails, as here as well, null is not compliant with the Number interface.

This might not be comprehensive, therefore, we should check all methods, that are yet implement in the abstract class.

@ckittl ckittl added the bug Something isn't working label May 18, 2021
@ckittl ckittl self-assigned this May 18, 2021
@ckittl ckittl changed the title Override "toString" in EmptyQuantity API-alignment bugs with EmptyQuantity May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant