Skip to content
This repository has been archived by the owner on Dec 23, 2020. It is now read-only.

Point3d.Unset should have a special string ovevrride #184

Open
goswinr opened this issue Aug 14, 2018 · 1 comment
Open

Point3d.Unset should have a special string ovevrride #184

goswinr opened this issue Aug 14, 2018 · 1 comment

Comments

@goswinr
Copy link

goswinr commented Aug 14, 2018

When calling ToString() on Point3d.Unset it prints
-1.23432101234321E+308,-1.23432101234321E+308,-1.23432101234321E+308
a string literal like
Point3d.Unset
or
Point3d.Unset (-1.2E+308, -1.2E+308, -1.2E+308)
would be nicer so that it is explicit that this is a special point value.
This helps in debugging too.
at line

public override string ToString()

@goswinr
Copy link
Author

goswinr commented Aug 14, 2018

I would in general add a type annotation and format any point as
Point3d(1,2,3)
instead of
1,2,3
but i understand that this might be a problem if someone relies on the current behaviour.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant