On 7/2/25 10:48 PM, Bagas Sanjaya wrote: > On 7/3/25 11:16, Randy Dunlap wrote: >> >> >> On 7/2/25 8:24 PM, Bagas Sanjaya wrote: >>> On Wed, Jul 02, 2025 at 10:24:47PM +0200, Nicolas Frattaroli wrote: >>>> diff --git a/Documentation/core-api/list.rst b/Documentation/core-api/list.rst >>>> new file mode 100644 >>>> index 0000000000000000000000000000000000000000..b0586056abb04d2bcc4518f7238ff9a94d3dd774 >>>> --- /dev/null >>>> +++ b/Documentation/core-api/list.rst >>>> @@ -0,0 +1,847 @@ >> >> [snip] >> >>> >>> The text LGTM, thanks! >>> >>> Reviewed-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx> >>> >> >> Please see https://people.kernel.org/tglx/notes-about-netiquette, >> especially "Trim replies". >> > > I did quote the whole diff to provide review context... > If you had commented on any specific lines in the patch, then that would be necessary. Even if you were commenting on only a few lines of the patch, most of the patch lines could be snipped out. The Subject: gives us the review context. It's pointless & annoying to other readers to have to scroll down thru an 847-line patch to see any kind of Tagged-by: reply. thanks. -- ~Randy
