minor patch related to save()

[Kris Thielemans]

Hi

Attached is a patch file with 2 tiny fixes related to the save() function:

load-save.cc: mention save_header_format_string in the see-also doc-string
for save()

pt-assign.cc: change  "default_save_format" to "default_save_options" in
former_built_in_variables (I think that was a typo as default_save_format
doesn't exist).

More info on why is below in the original thread on help-octave. I'd
appreciate it if somebody could check that in (if it's a good patch).

Thanks

Kris Thielemans
Algorithms and Software Consulting Ltd (http://asc.uk.com)

----------------------------------

On Mon, Dec 12, 2011 at 10:46 AM, Kris Thielemans
<kris.f.thielemans@ > wrote:
> Hi
>
> I was confused by the usage of default_save_options. I assigned a value to
> the variable, as opposed to calling the function. This is of course
clearly
> documented ("help default_save_options" tells you), but nevertheless I
only
> found out when trying to assign a value to save_header_format_string.
Doing
> that generates a clear warning about function vs variable. I wonder
> therefore if it wouldn't be helpful to issue the same warning for
> default_save_options. This would be easy to do (just add it to
> former_built_in_variables in pt-assign.cc).
>
> Another minor suggestion is to mention save_header_format_string in the
> documentation of save() (e.g. as "see also"). I only found out about it by
> checking the code.
>
> Many thanks
>
> Kris
>
>
>
> _______________________________________________
> Help-octave mailing list
> address@hidden
> https://mailman.cae.wisc.edu/listinfo/help-octave

Hi Kris,

Maybe you want to do it yourself and send a changeset to the
maintainer list? Devs are very busy preparing for 3.6 release and hey
may not take care of this soon.

If you can't do it, then please do fill a bug report (if this is a
minor bug in documentation and warning) so the issue is not forgotten.

https://savannah.gnu.org/bugs/?group=octave

Thanks

-- 
M. Sc. Juan Pablo Carbajal
-----
PhD Student
University of Zürich
http://ailab.ifi.uzh.ch/carbajal/

octave.save.patch
Description: Binary data