GNU bug report logs - #64202
[PATCH] Gnus: Add back end for Atom feeds (nnatom)

Previous Next

Full log

Message #11 received at 64202 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Daniel Semyonov <daniel <at> dsemy.com>, Andrew Cohen <cohen <at> bu.edu>,
 Eric Abrahamsen <eric <at> ericabrahamsen.net>,
 Stefan Monnier <monnier <at> iro.umontreal.ca>,
 Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 64202 <at> debbugs.gnu.org
Subject: Re: bug#64202: [PATCH] Gnus: Add back end for Atom feeds (nnatom)
Date: Wed, 21 Jun 2023 16:02:27 +0300

> Date: Wed, 21 Jun 2023 10:08:51 +0300
> From:  Daniel Semyonov via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
> 
> This patch adds a back end for Atom feeds to Gnus, and documents it.

Thanks for working on this.

I don't use Gnus, so my comments will be mostly to the documentation
parts.  I expect others (CC'ed) to review the code more thoroughly.

> +Some web sites provide an Atom Syndication Format feed.  Atom is a web
> +feed format similar in function to RDF Site Summary (@xref{RSS}).
                                                        ^^^^^^^^^^
This should be @pxref, not @xref.

> +Note, however, that the server address shouldn't be prefixed with
> +@code{http://} or @code{https://}.

These should use @file, not @code, unless most or all of the rest of
the Gnus manual uses @code for URLs.

> +The @code{nnatom} back end saves a server data file in the
> +@code{atom} sub-directory of @code{gnus-directory} for each feed.

These should definitely be @file, not @code, as these are file names.

> +@defmac nnatom-define-basic-backend-interface backend
> +Define server variables expected by @code{nnatom} and import its back
> +end functions for BACKEND.  This macro (or code similar to it) should
                     ^^^^^^^
This should be @var{backend}.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.