prisonerjohn / processing-web

0 stars 0 forks source link

[CLOSED] Reference generator script parses same-name methods incorrectly #137

Open prisonerjohn opened 10 years ago

prisonerjohn commented 10 years ago

Issue by benfry Tuesday May 14, 2013 at 20:52 GMT Originally opened as https://github.com/processing/processing-web/issues/137


Original report: https://github.com/processing/processing/issues/1520

prisonerjohn commented 10 years ago

Comment by REAS Tuesday Mar 25, 2014 at 04:17 GMT


This is a good place to see the issue: http://processing.org/reference/mouseDragged_.html

In the source, we have this:

   * @webref input:mouse
   * @see PApplet#mouseX
   * @see PApplet#mouseY
   * @see PApplet#mousePressed
   * @see PApplet#mousePressed()
   * @see PApplet#mouseReleased()
   * @see PApplet#mouseMoved()

But in the final reference file, there are two links to mousePressed and none to mousePressed()

prisonerjohn commented 10 years ago

Comment by sansumbrella Tuesday Mar 25, 2014 at 14:45 GMT


Javadoc can tell the difference between fields and methods, so it should be able to parse the above and give us the results we want. Even leaving out the @see tag for the field, though, I end up getting linked to the field, not the method. Poking around in the generator code, it's possible that we could catch this in our getName() method.

I'll look at this once I can build the reference generator on my machine again; switching to Mavericks broke ant compiling. Would be happy for advice on how to point ant to the javadoc packages in this brave new world.

prisonerjohn commented 10 years ago

Comment by sansumbrella Sunday Apr 06, 2014 at 16:34 GMT


Fixed in dbc506e89e64e8a37b6cdd3d6f00adced08e237b

prisonerjohn commented 10 years ago

Comment by alignedleft Sunday Apr 06, 2014 at 16:46 GMT


Fantastic, thank you, David!