[CLJ-792] Refactor method resolution code out of Compiler and into Reflector Created: 11/May/11 Updated: 20/Feb/12 |
|
| Status: | Open |
| Project: | Clojure |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major |
| Reporter: | Alexander Taggart | Assignee: | Alexander Taggart |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Attachments: |
|
| Description |
|
Issues:
By consolidating the duplicated code, moving the reflection-related parts into Reflector, and providing a straightforward API, it should be easier to read and understand the method resolution process. Further, improvements to (e.g., CLJ-445) the mechanism for reflecting on class members can largely be isolated from the Compiler. And the few points of coordination (e.g., Compiler emitting same arg and return types as Reflector does when invoking) can be clearly identified and documented. |
| Comments |
| Comment by Stuart Sierra [ 17/Feb/12 2:28 PM ] |
|
Patch does not apply as of commit f5bcf64. |
| Comment by Alexander Taggart [ 17/Feb/12 3:14 PM ] |
|
Yeah, year-old patches tend to do that. |
| Comment by Andy Fingerhut [ 20/Feb/12 1:11 PM ] |
|
I don't know if this is helpful or not, but this updated version of Alexander's patch applies cleanly to latest Clojure head as of Feb 20, 2012. It compiles, but does not pass ant test. |