-
Notifications
You must be signed in to change notification settings - Fork 4
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
Replace reflection with generated code #29
Comments
Yeah, it's doable, I'd say. |
@sockeqwe we could use some pointers here 😄 |
What do you exactly want to delete with annotation processing? I have to dive deeper into your code next week to see what exactly could be eliminated with annotation processing, but from what I have read in the wiki:
The problem is, that annotation processing is a java thing. There is no build in functionality so that the annotation processor knows about your android projects Unfortunately this is means a painful setup for the user of this library. But there is more painful things coming. Basically, annotation processor doesn't know about product flavors etc. So the user would also have to pass that as an annotation processor argument to your annotation processor. But wait, there is more ... But wait, there is more .... TLDR I think this is a huge amount of work and not sure if it's worth the effort, but if you want to do this I think writing a Gradle Plugin is the better option because then you can access to android's gradle plugin and configurations. |
Thank you for the quick feedback, man. Looks tricky and huge work, indeed. @mrsasha we could decrease the priority of this one and point our efforts to other enhancements first. |
Thanks for this, @sockeqwe! I didn't actually think about how it would be done, but I thought that through annotation processing you get a complete access to all the files, and not just the Java sources. So it looks like gradle plugin would be the way to go. btw, @hamen this was never a very-high priority, more like a nice to have. we'll see. there's lots of other stuff that needs to be done :) |
Evaluate if this could actually be done... and do it.
The text was updated successfully, but these errors were encountered: