[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: minor patch related to save()
From: |
Ben Abbott |
Subject: |
Re: minor patch related to save() |
Date: |
Tue, 13 Dec 2011 11:33:24 -0500 |
On Dec 13, 2011, at 11:03 AM, Kris Thielemans wrote:
>
> 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
>>>
>> 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
>
> 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
To be sure this isn't forgotten, please enter it in the patch tracker.
https://savannah.gnu.org/patch/?group=octave
Ben