Re: [PATCH] Improve Gravatar support

[Lars Ingebrigtsen]

After skimming (on holiday), looks good to me.  Extremely minor comments
below:

"Basil L. Contovounesios" <address@hidden> writes:

>  (defcustom gnus-gravatar-too-ugly gnus-article-x-face-too-ugly
> -  "Regexp matching posters whose avatar shouldn't be shown automatically."
> -  :type '(choice regexp (const nil))
> +  "Regexp matching posters whose avatar shouldn't be shown automatically.
> +Nil means show all avatars."

We don't capitalise nil in documentation, do we?

> -       ;; The buffer can be gone at this time
> -       (when (buffer-live-p (current-buffer))
> +      ;; The buffer can be gone at this time.
> +      (when (buffer-name)

I think buffer-live-p is clearer than just saying buffer-name.

> -  (interactive (list t)) ;; When type `W D g'
> +  (interactive "p") ;; When type `W D g'

Perhaps grammer fix?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no