git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 4/4] convert: use size_t for text statistics and ident counts

From
Patrick Steinhardt <ps@pks.im>
Date
Nov 2, 2026, 11:09 UTC
Message-ID
<auhvUm4VwXyWNhsE@pks.im>
In-Reply-To
<20261031101619.172770-2-gitter.spiros@gmail.com>
On Sat, Oct 31, 2026 at 11:16:19AM +0100, Elia Pinto wrote:
> On Fri, Oct 30, 2026 at 12:44:40PM +0100, Patrick Steinhardt wrote:
[snip]
Show 12 quoted lines
> > These paragraphs very much read like AI to me. I would appreciate a
> > human touch, also to demonstrate that a human was involved between us
> > and the AI that tries to understand what was done.
> 
> You're completely right. I used AI tooling to help draft that commit
> message, and it definitely shows. Fair cop.
> 
> To clarify my involvement: I review the code, check the cases, run the
> tests, and take full responsibility for what gets submitted. But the
> wording here missed the mark. For v3 I rewrote the message myself,
> keeping it much shorter, focusing on a single example (lone LFs), and
> dropping the heavy-handed explanations.

Sorry, but is this reply to my ask to proof read what AI generates and make it sound less AI generated repeating the _exact same thing_ that I just asked you to change?

To put it bluntly: behaviour like this is what will make me ignore your patch series altogether going forward. It is frustrating to me, and I only have limited time to spend on reviews. So I will much rather spend my own time on patches where I don't feel like I'm talking to a meat proxy, because I'm a (somewhat) social creature that prefers to talk to humans.

So please reevaluate what you are doing here.
Patrick
Previous: Elia PintoNext: Junio C Hamano
Message 17 of 24 in “use size_t for a few more object and buffer sizes”
  1. 0/3 use size_t for a few more object and buffer sizesElia Pinto, Oct 21, 2026
  2. 1/3 odb: use size_t for the length of objects to writeElia Pinto, Oct 21, 2026
  3. Patrick SteinhardtOct 21, 2026
  4. 2/3 diff: use size_t for the allocation size of word-diff buffersElia Pinto, Oct 21, 2026
  5. Patrick SteinhardtOct 21, 2026
  6. 3/3 convert: use size_t for buffer sizesElia Pinto, Oct 21, 2026
  7. Patrick SteinhardtOct 21, 2026
  8. 0/4 use size_t for a few more object and buffer sizesElia Pinto, Oct 30, 2026
  9. 1/4 odb: use size_t for the length of objects to writeElia Pinto, Oct 30, 2026
  10. Patrick SteinhardtOct 30, 2026
  11. Elia PintoOct 31, 2026
  12. 2/4 diff: use size_t for the allocation size of word-diff buffersElia Pinto, Oct 30, 2026
  13. 3/4 convert: use size_t for buffer sizesElia Pinto, Oct 30, 2026
  14. 4/4 convert: use size_t for text statistics and ident countsElia Pinto, Oct 30, 2026
  15. Patrick SteinhardtOct 30, 2026
  16. Elia PintoOct 31, 2026
  17. Patrick SteinhardtNov 2, 2026
  18. Junio C HamanoNov 2, 2026
  19. Patrick SteinhardtNov 3, 2026
  20. 0/4 use size_t for a few more object and buffer sizesElia Pinto, Oct 31, 2026
  21. 1/4 odb: use size_t for the length of objects to writeElia Pinto, Oct 31, 2026
  22. 2/4 diff: use size_t for the allocation size of word-diff buffersElia Pinto, Oct 31, 2026
  23. 3/4 convert: use size_t for buffer sizesElia Pinto, Oct 31, 2026
  24. 4/4 convert: use size_t for text statistics and ident countsElia Pinto, Oct 31, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.