-
Notifications
You must be signed in to change notification settings - Fork 77
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
Better support for concrete values #451
Comments
I am now doing the first two, although I haven't done much to test the second. I need to look at the code and maybe refactor a bit, though, I added support for creating and using the constructors in with the existing constructor support code, but this means there are now more cases and probably a bit more duplication as well. |
@mahills sounds fantastic to me! Thanks a lot. I will started testing immediately. |
I have now implemented field selection on concrete values and that works fine. Subscription is not yet behaving as it should. Consider:
gives
|
I'll try to get this added over the next couple of days. |
This one still seems to be open, I still get the above error message. |
Ah, my "next couple of days" apparently didn't work out quite as expected :( I'll take a look -- I'm assuming subscripts ignore layout, like the constructors do? |
Yes, I think also subscripts should ignore layout. On Mon, Jan 13, 2014 at 12:37 AM, mahills notifications@github.com wrote:
Jurgen Vinju
|
Indeed! |
Just to verify, the correct behavior should be:
Is this all correct? |
Yes, and all results are of type Tree, necessarily— On Tue, Jan 14, 2014 at 1:35 AM, mahills notifications@github.com wrote:
|
@mahills, here is my take on this:
|
A yes! We can be more precise with lists. Cool.— On Tue, Jan 14, 2014 at 8:44 AM, Paul Klint notifications@github.com
|
Overall goal: provide better support for concrete values:
The compiler does not support any of the above yet.
The type checker needs to:
The text was updated successfully, but these errors were encountered: