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
Adding tests for ufcs fundamental types #690
Adding tests for ufcs fundamental types #690
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a problem with integer and float types here: If you have a smaller integer, like short or byte, it can get implicitly upcast into an int
. So we should probably change the type matching algorithm to somehow catch these implicit casts. (e.g. a function dsymbol implicitly casts into other dsymbol)
I think for alias this'd structs we need to cover that as well.
@@ -2,4 +2,4 @@ set -e | |||
set -u | |||
|
|||
../../bin/dcd-client $1 -c100 -I"$PWD"/fooutils file.d > actual.txt | |||
diff actual.txt expected.txt | |||
diff actual.txt expected.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed last empty line here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what you mean, by this comment.
you are right, I haven't thought about that. |
Included implicit integer upcast to matching algorithm of UFCS. |
I added more tests for UFCS and updated readme according to types