[Octave-patch-tracker] [patch #9082] some doxygen

[John W. Eaton]

Follow-up Comment #2, patch #9082 (project octave):

I checked in an additional change that clarifies the doc strings.

Also, please don't make changes like this:


@@ -4778,7 +4783,9 @@ feval (const std::string& name, const oc
   octave_value fcn = symbol_table::find_function (name, args);
 
   if (fcn.is_defined ())
-    retval = fcn.do_multi_index_op (nargout, args);
+    {
+       retval = fcn.do_multi_index_op (nargout, args);
+    }
   else
     {
       try


If the code following if or else is just one statement, then it doesn't need
to be surrounded by braces and our style is to almost always omit them.

    _______________________________________________________

Reply to this item at:

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

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