[Conkeror] Bug: User variable default value reported incorrectly
Nicholas A. Zigarovich
nicktastic at gmail.com
Mon Sep 15 06:51:49 PDT 2008
Apathy nothing. You worked hard on conkeror all day long on Friday and
deserved a break (and then you worked some more).
You and jbms convinced me; I vote 1.
- Nick
On Sun, Sep 14, 2008 at 7:23 PM, John J Foerch <jjfoerch at earthlink.net> wrote:
> Nick <nicktastic at gmail.com> writes:
>> Reported by: John J. Foerch <jjfoerch -at- earthlink -dot- net>
>> Date: September 12, 2008
>> Status: Open
>>
>> Description:
>>
>> The default value of variables defined with define_variable is
>> remembered for use by the help system; however if the variable is
>> changed the default value is incorrectly reported. This may only apply
>> to certain types of variables.
>>
>> 1. Change one of the default MIME type external handlers:
>> define_mime_type_external_handler("application/pdf", "xpdf");
>> 2. C-h v mime_type_external_handlers
>> 3. Notice that the default listed for /^application\/pdf$/ is xpdf
>> when it should be evince.
>>
>> Discussion:
>>
>> < retroj`> I shouldn't spend more time on conkeror today :) :(
>
> Hey thank you for making sure my apathy is known far and wide. :)
>
> I will just follow up with the *real* discussion that did take place
> about this bug.
>
> The problem concerns user variables that are objects (or arrays, etc)
> instead of scalars. When the object is modified, rather than wholy
> replaced by a different object, the stored default value still points to
> the original, modified object.
>
> Here are our options:
>
> 1) Don't store default values for object user variables.
>
> 2) Serialize the default values and store them as strings.
>
> I am more in favor of option 1, but then again, I don't see the point
> in even storing default values for anything in the first place.
>
> Votes?
>
> --
> John Foerch
>
>
> _______________________________________________
> Conkeror mailing list
> Conkeror at mozdev.org
> https://www.mozdev.org/mailman/listinfo/conkeror
>
More information about the Conkeror
mailing list