X-Git-Url: http://cgit.sxemacs.org/?p=gnus;a=blobdiff_plain;f=lisp%2Fgravatar.el;h=ffbc37ae15887748346f35d7c2aa433a5434dc4d;hp=a10ea21b96b86e7da950c01c494d4243a08832b7;hb=a0193b94630db19a690256aa42efbb0407734af1;hpb=d979dbd7660096bd7835773b4334281e6868fa5c diff --git a/lisp/gravatar.el b/lisp/gravatar.el index a10ea21b9..ffbc37ae1 100644 --- a/lisp/gravatar.el +++ b/lisp/gravatar.el @@ -1,6 +1,6 @@ ;;; gravatar.el --- Get Gravatars -;; Copyright (C) 2010-2012 Free Software Foundation, Inc. +;; Copyright (C) 2010-2014 Free Software Foundation, Inc. ;; Author: Julien Danjou ;; Keywords: news @@ -33,19 +33,25 @@ :group 'comm) (defcustom gravatar-automatic-caching t - "Whether cache retrieved gravatar." + "Whether to cache retrieved gravatars." + :type 'boolean :group 'gravatar) +;; FIXME a time value is not the nicest format for a custom variable. (defcustom gravatar-cache-ttl (days-to-time 30) "Time to live for gravatar cache entries." + :type '(repeat integer) :group 'gravatar) +;; FIXME Doc is tautological. What are the options? (defcustom gravatar-rating "g" "Default rating for gravatar." + :type 'string :group 'gravatar) (defcustom gravatar-size 32 "Default size in pixels for gravatars." + :type 'integer :group 'gravatar) (defconst gravatar-base-url @@ -103,6 +109,8 @@ If no image available, return 'error." (gravatar-create-image data nil t) 'error))) +(autoload 'help-function-arglist "help-fns") + ;;;###autoload (defun gravatar-retrieve (mail-address cb &optional cbargs) "Retrieve MAIL-ADDRESS gravatar and call CB on retrieval. @@ -130,9 +138,7 @@ You can provide a list of argument to pass to CB in CBARGS." "Retrieve MAIL-ADDRESS gravatar and returns it." (let ((url (gravatar-build-url mail-address))) (if (gravatar-cache-expired url) - (with-current-buffer (if (featurep 'xemacs) - (url-retrieve url) - (url-retrieve-synchronously url)) + (with-current-buffer (url-retrieve-synchronously url) (when gravatar-automatic-caching (url-store-in-cache (current-buffer))) (let ((data (gravatar-data->image)))