The current name binding definition handles class names, labels, and local variable indices. You should extend this with name resolution for method and field names and with typical constraints.

Submitted by Guido Wachsmuth on 14 February 2014 at 12:37

On 14 February 2014 at 12:37 Guido Wachsmuth removed tag @guidowachsmuth

On 14 February 2014 at 12:38 Guido Wachsmuth tagged mdsd14

On 14 February 2014 at 12:38 Guido Wachsmuth removed tag feature

On 14 February 2014 at 12:38 Guido Wachsmuth tagged improvement

On 21 February 2014 at 09:45 Jeff Smits tagged @jeffsmits

On 21 February 2014 at 14:35 Jeff Smits commented:

Method and field names are now being resolved on my branch. But the AST is changed (desugared) for it to work nicely in NaBL, so that may break other things.
Don’t have warnings/errors yet. That’s still TODO.


On 28 February 2014 at 09:15 Jeff Smits tagged 0.1

On 4 March 2014 at 20:30 Guido Wachsmuth commented:

This might work w/o desugaring, if you make the descriptor a property of a name and use this for references as well:

...: defines Method n of descriptor d
...: refers to Method n of descriptor d

On 5 March 2014 at 09:29 Jeff Smits commented:

Good idea. I didn’t remember how to add those kinds of contraints in NaBL, but using properties makes sense.


On 7 March 2014 at 09:05 Jeff Smits commented:

Ok, I tried the property constraint approach and it works. Thanks!


On 7 March 2014 at 10:13 Jeff Smits commented:

Done with the extra constraints. Only need pretty-printing to improve error messages.
Will wait for #5 to complete so pretty-printing can be used.


On 24 April 2014 at 16:26 Jeff Smits commented:

Ok, used the pretty-printing. A bug that cost a lot of time was that <init> (that’s a Jasmin special method name) would disappear from any error message. Eventually found out that Eclipse allows HTML in those tooltip messages for markup. So I had to HTML-escape some error messages…
I opened a pull-request on GitHub.


On 15 May 2014 at 19:23 Jeff Smits commented:

PR was merged


On 15 May 2014 at 19:23 Jeff Smits closed this issue.

Log in to post comments