[Octave-patch-tracker] [patch #9056] documentation for Java methods and fieldnames

[Ernst Reissner]

Follow-up Comment #3, patch #9056 (project octave):

Hi Mike, 
Well done, but I have still some comments on the comments of fieldnames: 
- You missed the description if a struct is given. 
- I missed the information that for java 
  only the member fields are returned. 
The latter may be a bug. 
Is there someone with a new version of matlab to try? 
Maybe just have a look at 'help fieldnames'.
Try: 
octave:1> fieldnames('org.octave.ClassHelper')
ans = {}(0x0)
although scripts/java/org/octave/ClassHelper.java 
clearly shows a static field (loader). 

Whether bug or not, the behavior shall be documented. 

With your version of methods.m i am not content. 
If you hava a look at the code, 
there are three cases which shall be clearly commented: 
- octave-object
- octave-classname 
- java-object 
- java-classname. 

This was done with my comments but not with yours. 
Also part of my rewording comes from my intent to unify the two comments. 

For methods.m I made the mistake to say that the NAMES of the public methods
are returned, 
In fact it is the SIGNATURE without access modifier. 

I also forgot to comment, that for java only the MEMBER methods are returned.

As for fieldnames, I suppose it is a bug or a deficiency 
not to display the static methods. 


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?9056>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/