Clojure

clojure.lang.Reflector.invokeMatchingMethod is not complete (rejects pontentially valid method invocations)

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

There exists invoke expressions on instances, where Java is able to perform the call, yet clojure is not reflectively.
The problem is when the declaringClass of the found method is not public then the call to getAsMethodOfPublicBase uses the found method or searching for classes/interfaces that contain/define this method yet are declared publicly.

This restricts the possible search space. I suggest that if target is not null (e.g. is not a static method), the target.getClass() should be used instead as a root for getAsMethodOfPublicBase.
This fixes my issue.

Activity

Hide
Assembla Importer added a comment -
Show
Assembla Importer added a comment - Converted from http://www.assembla.com/spaces/clojure/tickets/259
Hide
Assembla Importer added a comment -

richhickey said: How about some sample case that demonstrates the problem?

Show
Assembla Importer added a comment - richhickey said: How about some sample case that demonstrates the problem?
Hide
Assembla Importer added a comment -

hiredman said: Related association with ticket #126 was added

Show
Assembla Importer added a comment - hiredman said: Related association with ticket #126 was added
Alex Miller made changes -
Field Original Value New Value
Fix Version/s Approved Backlog [ 10034 ]
Fix Version/s Backlog [ 10035 ]
Alex Miller made changes -
Fix Version/s Backlog [ 10035 ]
Alex Miller made changes -
Priority Major [ 3 ]
Hide
Alex Miller added a comment -

Better description, case, and patch in new ticket CLJ-1609.

Show
Alex Miller added a comment - Better description, case, and patch in new ticket CLJ-1609.
Alex Miller made changes -
Resolution Duplicate [ 3 ]
Status Open [ 1 ] Closed [ 6 ]

People

  • Assignee:
    Unassigned
    Reporter:
    Anonymous
Vote (2)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: