search

DeveloperLiberationFront / Program-Navigation-Plugin

2 stars 3 forks source link

Update code to highlight variables instead of statements #4

Closed jssmith1 closed 8 years ago

chbrown13 commented 8 years ago

Got SimpleName to highlight all identifiers (variables, method names, classes, etc.). Modify to only allow variables: http://grepcode.com/file/repository.grepcode.com/java/eclipse.org/4.3.1/org.eclipse.jdt/core/3.9.1/org/eclipse/jdt/core/dom/SimpleName.java#SimpleName

chbrown13 commented 8 years ago

Can't extend SimpleName outside of package

chbrown13 commented 8 years ago

Local version works for declared variables and parameters (highlights only selected occurrence). How should we handle for and while loops @jssmith1?

chbrown13 commented 8 years ago

Done

CaptainEmerson commented 8 years ago

@chbrown13 : it'd be helpful to me if, when you commit, you could mention the issue, so the code and the issue get linked together. It'll help add some transparency.

chbrown13 commented 8 years ago

Ok

On Mon, Jan 4, 2016 at 3:01 PM, Emerson Murphy-Hill < notifications@github.com> wrote:

@chbrown13 https://github.com/chbrown13 : it'd be helpful to me if, when you commit, you could mention the issue, so the code and the issue get linked together. It'll help add some transparency.

— Reply to this email directly or view it on GitHub https://github.com/DeveloperLiberationFront/Program-Navigation-Plugin/issues/4#issuecomment-168786561 .