11 pointsby itvision4 hours ago3 comments

Alifatisk3 hours ago
The dialog was something to read. Doesn’t sound like ai but rather something a developer has practiced for years and no one has challenged the dev on this, until now.
Also, there is discussions on what a commit message should contain, apparently the patch had user guidance in the commit message. Literally on how to install dependencies such as gcc using pacman, in the commit message.
https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21595
dryarzeg4 hours ago
const int EIGHT = 8 lol
I really doubt any AI (even some small local models) would actually generate something like this :)
- SomeUserName4322 hours ago
  I've run into something akin to `const int EIGHT = 7`.
  Courtesy of TCS.
- Alifatisk3 hours ago
  Yeah, I don’t think this is a case of ai slop. LLMs tend to be verbose with the comments but are fine with magic constants, at least from my experience.
high_na_euv3 hours ago
I dont see problem with user manual in commit as long as reasoning for the commit is clearly written
- potbelly8313 minutes ago
  yeah, they seemed to be nitpicking there, better to focus attention on what someone's actually trying to commit
- mort963 hours ago
  Really? You think a git commit message in a C project is the right place to document how to install make and gcc on arch linux? That's not better suited as part of a readme file?
  - high_na_euv2 hours ago
    It should be in readme but I wouldn't complain if commit with new feature also had it
    2 hours ago
    undefined