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

Simple Generics Support #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

jcmm33
Copy link

@jcmm33 jcmm33 commented Oct 25, 2012

Introduced simple support for generics, allowing for methods of classes
and statics to be invoked with generic types (e.g. Data.Get(100)).

Simplistric test added to validate operation of generics support.

It should be noted that whilst all tests run correctly these have only been performed against the vici.core solution running under vs.net 2012. No changes have been made to projects for differing platforms e.g. monotouch,monodroid, wp 7.

Two new files were introduced 1 for the parsing and 1 for the generic variable expression.

Introduced simple support for generics, allowing for methods of classes
and statics to be invoked with generic types.
Introduced concept of a simple function callback to be used (when
specified) to lookup unresolved variable names, allowing for resolving
of variables without neccessarily adding them into the context.
@activa
Copy link
Contributor

activa commented Feb 13, 2013

It's very hard to verify this patch because tabs were converted to spaces or the other way around. Or CR/LF line endings were. Either way, this means that Git thinks that all lines have changed.

The patch is very useful though, so I will check the changes manually when time permits.

@jcmm33
Copy link
Author

jcmm33 commented Apr 19, 2013

Hmm wonder why the CR/LFs are like that - don't remember doing anything specific. Wonder if its because I used VS.NET 2012 ?

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

Successfully merging this pull request may close these issues.

2 participants