groovy-eclipse icon indicating copy to clipboard operation
groovy-eclipse copied to clipboard

Add support for code hover/select of method name with spaces, etc.

Open eric-milles opened this issue 7 years ago • 2 comments

Consider the following:

def "some method name for whatever reason"() {
}

Code hover/select (F2/F3) should treat entrie method name string as one entity. Currently selection has been disabled in org.codehaus.groovy.eclipse.codebrowsing.requestor.CodeSelectHelper.isStringLiteral(ASTNode, char[], int, int). With the extra clause removed from "isStringLiteral", selection takes place on individual words of the method name only.

eric-milles avatar Jul 17 '18 15:07 eric-milles

Does really Groovy support something like that? :-O

mauromol avatar Aug 10 '18 06:08 mauromol

Yes indeed. Any walkthrough of the Spock framework for testing shows method names in this fashion. I think testing is the primary use case for the feature. Although a.getAt("b"), a["b"] and a."b" all work fine in case you did this in general practice.

eric-milles avatar Aug 10 '18 14:08 eric-milles