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

Unable to use TypeAccessor with IL Generated type. #59

Open
guy-lud opened this issue Jun 6, 2018 · 8 comments
Open

Unable to use TypeAccessor with IL Generated type. #59

guy-lud opened this issue Jun 6, 2018 · 8 comments

Comments

@guy-lud
Copy link

guy-lud commented Jun 6, 2018

Hi Marc,

I have a Lib named SimpleConfig which is like IOptions.
I thought of using FastMemeber to check performance issues.
To make a long story short i am creating a Type from an interface and when trying to use FastMemeber on a type from a dynamic assembly i get not supported exception, when trying to use it on the interface type when inheritance takes place i get outOfRangeException,

Can we use a flat hierarchy ?

@mgravell
Copy link
Owner

mgravell commented Jun 7, 2018 via email

@guy-lud
Copy link
Author

guy-lud commented Jun 7, 2018

Sure, I'll create a repo tonight (my night) and i will post it asap

@guy-lud
Copy link
Author

guy-lud commented Jun 7, 2018

Hi Marc,

I've created a branch with a demo of the problem, please take a look at this. The Test name is Marc_Demo

@guy-lud
Copy link
Author

guy-lud commented Jun 17, 2018

Hi @mgravell,

Did you have the time to look at it please ?

@mgravell
Copy link
Owner

mgravell commented Jun 17, 2018 via email

@guy-lud
Copy link
Author

guy-lud commented Jun 17, 2018

@mgravell thanks for replying :)

@tippmar
Copy link
Contributor

tippmar commented Sep 25, 2018

This sounds pretty similar to the issues I found when working with interfaces. I've forked the fast-member repo and implemented a fix for the issue - you might take a look and see if it helps you or not: https://github.com/tippmar/fast-member -- I've also created a PR to this but it hasn't been merged yet.

@guy-lud
Copy link
Author

guy-lud commented Sep 26, 2018

@tippmar, I'll take a look.
Thanks

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

No branches or pull requests

3 participants