[Conkeror] Bug: User variable default value reported incorrectly

[John J Foerch]

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