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

routeMacro refactoring #45

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

routeMacro refactoring #45

wants to merge 3 commits into from

Conversation

Voltir
Copy link

@Voltir Voltir commented Nov 28, 2015

This changes routeMacro to use a tree based datastructure, hopefully avoiding a JVM limitation when a method gets too large. In theory its also higher performant, but I haven't done any empirical testing.

@Voltir
Copy link
Author

Voltir commented Apr 1, 2016

Poke. Im still using my fork of this repo and id like to update the dependencies.

@Daxten
Copy link

Daxten commented Jun 3, 2017

Hi @Voltir ,
I talked to lihaoyi and he isn't active on this project at the moment. He said that he wouldn't mind if I fork/release it, which I did: https://github.com/Daxten/autowire

would you mind to either make a pull request there and/or tell me if it's fine for you if I merge this?

best regards

@Voltir
Copy link
Author

Voltir commented Jun 4, 2017

I think its fine if you merge it - probably needs to be brought up to date.

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