#685 JavaDoc fixes for master

Accepted
closed
nobody
None
master
1
2013-10-14
2013-09-29
Egon Willighagen
No

Most is just inheriting documentation from super classes and interfaces, but also other stuff:

https://github.com/egonw/cdk/commits/474-m-javaDocFixes

Six patches.

Discussion

  • John May
    John May
    2013-09-30

    Looks good, applied and pushed. Couple of comments - I'll add another commit removing the extra '}'. Also I wouldn't overuse {@link} - if the class is in the method parameters or return type it will be linked already in that HTML paragraph somewhere. More {@link} mean more to fix when things move :-).

     
  • Umm... right... generally OK, but:

    test-fixes.log | 1852 ++++++++++
    test.log | 1859 ++++++++++
    types-patch.log | 3558 ++++++++++++++++++++

     
  • John May
    John May
    2013-10-07

    ?

     
  • John May
    John May
    2013-10-07

    Oh right - my mistake.

     
    • status: open --> closed
    • Group: Needs_Review --> Accepted
     
  • Applied and pushed.