From: Chris Travers [mailto:chris.trav...@adjust.com] > On Fri, Feb 8, 2019 at 9:40 AM Tsunakawa, Takayuki > <tsunakawa.ta...@jp.fujitsu.com> wrote: > I'm sorry if I repeat what someone proposed in the past, but it > would be nice if the CF app could show the number of modified lines (added > + deleted) for each patch, e.g., the last line of diffstat command output. > That would help young newbies to choose patches. > > > > An important prerequisite here would be to list all patches on the most > recent email. Right now it just lists one so you have to follow the link > to the hackers list entry and look at each patch one at a time. > > So if we could grab all attachments ending in .diff or .patch and list line > counts, that would be a welcome improvement.
I understood it may be difficult to extract all patch files in a single email. Sorry, I can't tell whether it's possible... Another idea is to add the number of lines of the mail thread. Reviewers can expect that a patch with short discussion may be easy to work on, although it's not always true of course. Long mail threads are hard to understand for those who are new and/or not good at English. Regards Takayuki Tsunakawa
