[Conkeror] [PATCH 1/2] webjump: reworked code for the "alternative" url

John J Foerch jjfoerch at earthlink.net
Sat Feb 7 12:05:47 PST 2009


David Kettler <kettler at internode.on.net> writes:
> The alternative url (which is used when no argument is supplied by the
> user when invoking a webjump) should now be specified explicitly using
> the $alternative keyword to define_webjump().  The previous method
> (which will continue to be supported) of specifying the handler as an
> array of two strings was a bit arbitrary.  Furthermore, it seems quite
> peculiar when considering an expansion of alternative urls for function
> handlers.
>
> Prepared for allowing search engine webjumps to specify an alternative
> url by allowing function handlers to have an alternative url.
>
> Expanded the comment describing the parameters to define_webjump().
>
> The supplied webjumps that have alternative urls are changed to use
> the new style (although the old style will still work).

This looks good.  I don't see any reason to keep compatibility with the
two-element-array form, though.  Conkeror's API is still unstable and I
think everybody understands they will have to update their rc's with
changes as we figure things out.

-- 
John Foerch



More information about the Conkeror mailing list