Re: PATCH: Make nnrss fail gracefully when it can't fetch a feed

[Katsumi Yamaoka <yamaoka@jpl.org>]

>>>>> In <87y82e1gg2.fsf@stone.tss.usg.edu> Mark Plaksin wrote:

> When nnrss can't fetch a feed an error is signalled.  This patch makes
> nnrss print a warning instead.

> When you use 'g' from the Group buffer, the current behavior prevents the
> rest of your feeds from being checked.  The root cause is the use of
> `(error)' in mm-url-insert-file-contents-external.  This patch just adds a
> condition-case in nnrss.el.

> -	  (mm-url-insert url)))
> +          (condition-case nil
> +              (mm-url-insert url)
> +            (error (nnheader-message 3 "nnrss: Failed to fetch %s" url)))))

Isn't it better to use just `message' since it is obviously an
error?  And also (sit-for N) might be required.

> I don't know whether I used the right -message function.  What's the
> difference between nnheader-message and gnus-message?

AFAIK, gnus- modules, mm- modules, nn- modules, and others are
mutually independent.  For example, there might be people who
use nn- modules for the application other than Gnus. (They
depend each other in practice, though.)

BTW, you'd better do the paper work for the copyright assignment
on Emacs (or Gnus) because the remainder of your `tiny change'
quota will be used up sooner or later. ;-)