Re: error trying to customize browse-url-browser-function
Hrvoje Niksic <hniksic(a)arsdigita.com> writes:
John Paul Wallington <jpw(a)shootybangbang.com> writes:
> Simon Josefsson <jas(a)extundo.com> wrote:
>
>> Jan, Hrvoje, what do you think of the patch below? It adds the alist and
>> plist widget from Emacs. The alist widget solve's browse-url.el's
>> problem, I did not test the plist widget but it looks OK.
>
> Hrvoje already added a plist widget to cus-edit.el:
[...]
Indeed. If I remember correctly, the plist widget I added was written
by Per Abrahamsen (which my ChangeLog entry fails to mention, shame on
me) and published on the custom list. I hope the new widget is
compatible with that.
The Emacs plist widget looks like:
(define-widget 'plist 'list
"A property list."
:key-type '(symbol :tag "Key")
:value-type '(sexp :tag "Value")
:convert-widget 'widget-plist-convert-widget
:tag "Plist")
while the XEmacs plist looks like:
(define-widget 'plist 'list
"A property list."
:match (lambda (widget value)
(valid-plist-p value))
:convert-widget 'custom-plist-convert-widget
:tag "Property List")
so I guess they aren't compatible. I guess it would be possible to
merge the definitions though.
[jas＠localhost xemacs-packages]$ egrep ":type .*plist" */*.el
[jas＠localhost xemacs-packages]$
Maybe noone uses the plist widget though.
Btw, other occurances of alist widget:
gnus/gnus/lisp/gnus-art.el: :type 'alist)
liece/lisp/liece-url.el: :type 'alist
net-utils/feedmail.el: :type 'alist
net-utils/feedmail.el: :type 'alist
Possible hit:
bbdb/lisp/bbdb-print.el: :type 'bbdb-print-alist-widget)