From: Richard Weinberger <richard.weinberger@gmail.com>
To: Wang Qing <wangqing@vivo.com>
Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Adaptec OEM Raid Solutions <aacraid@microsemi.com>,
"James E.J. Bottomley" <jejb@linux.ibm.com>,
"Martin K. Petersen" <martin.petersen@oracle.com>,
Luis de Bethencourt <luisbg@kernel.org>,
Salah Triki <salah.triki@gmail.com>,
David Woodhouse <dwmw2@infradead.org>,
Richard Weinberger <richard@nod.at>,
Ralf Baechle <ralf@linux-mips.org>,
"David S. Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>,
linux-mips@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
linux-scsi@vger.kernel.org, linux-mtd@lists.infradead.org,
linux-hams@vger.kernel.org, netdev@vger.kernel.org,
linux-decnet-user@lists.sourceforge.net,
Greg KH <gregkh@linuxfoundation.org>
Subject: Re: [PATCH 4/6] fs/jffs2: Delete obsolete TODO file
Date: Thu, 8 Apr 2021 00:20:17 +0200 [thread overview]
Message-ID: <CAFLxGvy7vKJHAms-QAyhcUiv58GYQy+zy3AZ3hP_g+tmh-X4xg@mail.gmail.com> (raw)
In-Reply-To: <1617087773-7183-5-git-send-email-wangqing@vivo.com>
On Tue, Mar 30, 2021 at 9:07 AM Wang Qing <wangqing@vivo.com> wrote:
>
> The TODO file here has not been updated for 14 years, and the function
> development described in the file have been implemented or abandoned.
>
> Its existence will mislead developers seeking to view outdated information.
Did you check whether all items in this list are really outdated?
Nobody shall ever blindly follow a TODO list without checking first which
points are still valid or not.
Removing that file does not magically solve the issues it describes.
--
Thanks,
//richard
next prev parent reply other threads:[~2021-04-07 22:20 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-30 7:02 [PATCH 0/6] Clean up obsolete TODO files Wang Qing
2021-03-30 7:02 ` [PATCH 1/6] mips/sgi-ip27: Delete obsolete TODO file Wang Qing
2021-03-30 13:07 ` Thomas Bogendoerfer
2021-03-30 7:02 ` [PATCH 2/6] scsi/aacraid: " Wang Qing
2021-03-30 7:02 ` [PATCH 3/6] fs/befs: " Wang Qing
2021-03-30 7:02 ` [PATCH 4/6] fs/jffs2: " Wang Qing
2021-04-07 22:20 ` Richard Weinberger [this message]
2021-03-30 7:02 ` [PATCH 5/6] net/ax25: " Wang Qing
2021-03-30 7:02 ` [PATCH 6/6] net/decnet: " Wang Qing
2021-03-31 0:00 ` [PATCH 0/6] Clean up obsolete TODO files patchwork-bot+netdevbpf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFLxGvy7vKJHAms-QAyhcUiv58GYQy+zy3AZ3hP_g+tmh-X4xg@mail.gmail.com \
--to=richard.weinberger@gmail.com \
--cc=aacraid@microsemi.com \
--cc=davem@davemloft.net \
--cc=dwmw2@infradead.org \
--cc=gregkh@linuxfoundation.org \
--cc=jejb@linux.ibm.com \
--cc=kuba@kernel.org \
--cc=linux-decnet-user@lists.sourceforge.net \
--cc=linux-hams@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-scsi@vger.kernel.org \
--cc=luisbg@kernel.org \
--cc=martin.petersen@oracle.com \
--cc=netdev@vger.kernel.org \
--cc=ralf@linux-mips.org \
--cc=richard@nod.at \
--cc=salah.triki@gmail.com \
--cc=tsbogend@alpha.franken.de \
--cc=wangqing@vivo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).