Вариант для распечатки		Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Изначальное сообщение		[ Отслеживать ]

"Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от opennews (??), 30-Янв-26, 13:12
Разработчики мультимедийного пакета FFmpeg попросили компанию AMD внимательнее относится подготовке  патчей и не отправлять сгенерированные через AI изменения без проведения ручного рецензирования. Недовольство вызвал набор патчей с реализацией возможности использования  AMD HIP SDK (Heterogeneous-compute Interface for Portability) на платформе Windows для ускорения обработки видео на системах с GPU AMD... Подробнее: https://www.opennet.me/opennews/art.shtml?num=64707
Ответить | Правка | Cообщить модератору

Сообщения

[Сортировка по времени | RSS]

1. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+35 +/–
Сообщение от Аноним (1), 30-Янв-26, 13:12
Вайбкодинг во всей красе, добро пожаловать в новый и прекрасный мир
Ответить | Правка | Наверх | Cообщить модератору

	2. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–6 +/–
	Сообщение от Аноним (-), 30-Янв-26, 13:17
	> Автор патча сообщил, что осознанно добавил инструкцию по использованию pacman, > написанную им 4 года назад > Вайбкодинг Какой вайбкодинг? 4 года назад еще никакого вайбкодинга не было.
	Ответить | Правка | Наверх | Cообщить модератору

	4. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+22 +/–
	Сообщение от expert (??), 30-Янв-26, 13:18
	> const int EIGHT = 8
	Ответить | Правка | Наверх | Cообщить модератору

	10. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от КО (?), 30-Янв-26, 13:26
	А это все спецы по шаблонному мышлению виноваты. Мол магические константы в коде антипаттерн - выносите определение в именованную константу. Ну чел и вынес. Ну и дал самое логичное имя.
	Ответить | Правка | Наверх | Cообщить модератору

	15. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+11 +/–
	Сообщение от Аноним (15), 30-Янв-26, 13:34
	Недавно похожий случай был. Один рецензент покритиковал мой код. Ему не понравилось, что счетчики цикла имеют очень короткие и неинформативные, с его точки зрения, имена - i, j, k.
	Ответить | Правка | Наверх | Cообщить модератору

	21. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от КО (?), 30-Янв-26, 13:37
	И как таких без вайбкодинга обходить?
	Ответить | Правка | Наверх | Cообщить модератору

	24. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+5 +/–
	Сообщение от nox. (?), 30-Янв-26, 13:39
	Конечно, вместо i надо было написать COUNTEROFTHEFIRSTCYCLE. Круто, чо.
	Ответить | Правка | К родителю #15 | Наверх | Cообщить модератору

	28. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+2 +/–
	Сообщение от User (??), 30-Янв-26, 13:44
	Не color, a colour! В смысле CounterOfTheFirstCycle ну или COUNTER_OF_THE... Остроконечники с тупоконечниками еще не договорились, приходите пожжа
	Ответить | Правка | Наверх | Cообщить модератору

	35. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от КО (?), 30-Янв-26, 14:04
	не то  не другое - у вас стили наименования не подходящие к переменным, могут быть путаницы с константами и классами. :)
	Ответить | Правка | Наверх | Cообщить модератору

	42. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+8 +/–
	Сообщение от Аноним (42), 30-Янв-26, 14:38
	Надо римскими числами: i, ii, iii, iv, v, vi,  ...
	Ответить | Правка | К родителю #24 | Наверх | Cообщить модератору

	56. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от Смузихлеб забывший пароль (?), 30-Янв-26, 16:37
	xxx :)
	Ответить | Правка | Наверх | Cообщить модератору

	57. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (57), 30-Янв-26, 16:40
	>Недавно похожий случай был. Один рецензент покритиковал мой код. Ему не понравилось, что раскиданные по двухэкранному с пятью уровнями вложенности счетчики цикла имеют очень короткие и неинформативные, с его точки зрения, имена - i, j, k.
	Ответить | Правка | К родителю #15 | Наверх | Cообщить модератору

	60. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (60), 30-Янв-26, 16:45
	имена нормальные, а вот 5 уровней вложенности на два экрана - бывает, конечно, что иначе никак, но обычно можно подразбить
	Ответить | Правка | Наверх | Cообщить модератору

	61. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от laindono (ok), 30-Янв-26, 16:56
	Яб добавил, что есть нюансы. Например если лексическое время жизни счётчика большое (длинный цикл), то я могу заменить i на index. Если эти счётчики имеют геометрический смысл, то заменю на x, y, z. И так далее. Но это эвристики, а не правила. Можно применять для себя, но нельзя заставлять других. Длинные имена в данном контексте могут ухудшать читабельность кода.
	Ответить | Правка | К родителю #15 | Наверх | Cообщить модератору

	84. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (84), 30-Янв-26, 18:48
	> Длинные имена в данном контексте могут ухудшать читабельность кода. Мне кажется, тут в другом дело. В математических формулах принято различные индексы обозначать одним символом. Поэтому в запрограммированных алгоритмах уместно поступать так же. Легче проверять (алгоритмы). Ну, и дополнительный бонус - компактность записи. Легче проверять (программу). Поэтому при составлении программы просто нужно знать меру. Иначе есть риск получить ... 1С. :)))
	Ответить | Правка | Наверх | Cообщить модератору

	68. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от premium user (?), 30-Янв-26, 17:36
	Щёчики не нужны, часто можно указатели на данные использовать вместо них, это экономит такты ЦП, регистры, и нейминг очевиден. (Pixel *pix = data, *pix_max = data + size; pix < pix_max; pix++) { pix->r = ...; }
	Ответить | Правка | К родителю #15 | Наверх | Cообщить модератору

	91. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (91), 30-Янв-26, 19:43
	> Щёчики не нужны, часто можно указатели на данные использовать вместо них, это экономит такты ЦП, регистры, и нейминг очевиден Ничего это не экономит, при косвенной адресации дополнительные регистры не занимаются. > (Pixel *pix = data, *pix_max = data + size; pix < pix_max; pix++) Нечитабельное уродство. По правде, обычно не нужны ни индексы, ни, тем более, указатели. В нормальных языках пишут ```c++ for (const auto& pixel: pixels) {} ``` ```rust for pixel in &pixels {} ```
	Ответить | Правка | Наверх | Cообщить модератору

	98. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от premium user (?), 30-Янв-26, 20:24
	> Ничего это не экономит, при косвенной адресации дополнительные регистры регистр под счётчик занимается > for (const auto& pixel: pixels) {} А когда из одного или более массивов читается, а в другой или несколько пишется? А с simd ускорением? Облом-с)
	Ответить | Правка | Наверх | Cообщить модератору

	102. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от Аноним (102), 30-Янв-26, 20:57
	>> Ничего это не экономит, при косвенной адресации дополнительные регистры > регистр под счётчик занимается А под переменную указателя pix не занимается? Ты в ассемблерный выхлоп заглядывал хоть когда-нибудь?
	Ответить | Правка | Наверх | Cообщить модератору

	107. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от premium user (?), 30-Янв-26, 21:37
	Было 2 регистра - щётчик и указатель - стал 1, и инкремент 1 вместо 2х, это так трудно понять?
	Ответить | Правка | Наверх | Cообщить модератору

	95. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от Аноним (102), 30-Янв-26, 20:10
	> Щёчики не нужны, часто можно указатели на данные использовать вместо них, это экономит такты ЦП, регистры, и нейминг очевиден Чел, загляни уже наконец хотя бы раз в ассемблерный выхлоп. Даже с отключенной оптимизацией. 🤦
	Ответить | Правка | К родителю #68 | Наверх | Cообщить модератору

	101. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от premium user (?), 30-Янв-26, 20:40
	Чтобы что? Ты каждую строчку на ассемблере перепроверяешь? Или просто веришь что компилятор сделает хорошо? Ну чекнул я щя void test(u8 *src, u8 *dst, int n) { for(int i = 0; i < n; i++) dst[i] = src[i]; } с -O2 - в ассемблере оно так и есть, под i счётчик выделился, на каждый цикл 3 сложения вместо 2х если юзать указатели. И если такая простая конструкция в 16 регистров x86 помещается, то как выше портянка 5 уровневой сложности - уже нет.
	Ответить | Правка | Наверх | Cообщить модератору

	103. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от Аноним (102), 30-Янв-26, 21:03
	> Чтобы что? Чтобы больше не писать тут чушь после того, как увидишь, что для цикла со счетчиком и твоего варианта с указателями получится идентичный машинный код. > Или просто веришь что компилятор сделает хорошо? Он гарантированно сделает хорошо в 95% случаев. А оставшиеся 5% - это оптимизации уж точно не уровня "заменил счетчик на указатель - и все стало летать".
	Ответить | Правка | Наверх | Cообщить модератору

	110. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от premium user (?), 30-Янв-26, 21:56
	> для цикла со счетчиком и твоего варианта с указателями получится идентичный машинный код. Чекнул, так и есть, gcc -O2 выходит слабоват в оптимизации, не зря кодеки на асме пишут. Ладно хоть код читабельнее.
	Ответить | Правка | Наверх | Cообщить модератору

	80. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от Аноним (-), 30-Янв-26, 18:29
	> Ну чел и вынес. Ну и дал самое логичное имя. К сожалению это кое-что говорит о его способностях к логическому мышлению и общему качеству кода. Ибо похабнее и глупее код даже AI не напишет. И это вовсе не комплемент двуногому програмеру. Если это лучшее описание для magic value которое вы можете родить - упс, вы вообще не программист.
	Ответить | Правка | К родителю #10 | Наверх | Cообщить модератору

	14. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (14), 30-Янв-26, 13:32
	Никаких магических переменных все ясно и понятно какие вопросы? Надо было const X = 8?
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	17. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+4 +/–
	Сообщение от КО (?), 30-Янв-26, 13:35
	const int TRINITY=8;
	Ответить | Правка | Наверх | Cообщить модератору

	89. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от oldskool (?), 30-Янв-26, 19:33
	// good luck with debug #define TRUE FALSE
	Ответить | Правка | Наверх | Cообщить модератору

	50. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+8 +/–
	Сообщение от ХрюХрю (?), 30-Янв-26, 16:05
	ничего вы не понимаете! надо const int BOCEMb = 8;
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	54. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+3 +/–
	Сообщение от онанист (?), 30-Янв-26, 16:14
	это чтобы в военное время заменить на 14
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	59. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от Аноним (60), 30-Янв-26, 16:43
	еще не факт, что это ИИ. Я вполне естественный "интеллект" видел, который воспринял "no magic numbers please" буквально и притащил "рефакторинг" вида const int THIRTY = 30.
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	76. Скрыто модератором	+/–
	Сообщение от Аноним (76), 30-Янв-26, 18:13
	Это скорее автор новости навайбкодил. https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21593/files
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	83. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от SVolf (?), 30-Янв-26, 18:41
	У меня на работе неиронично все константы и строковые литералы заставляют выносить в отдельные переменные, даже если оно 1 раз юзается. Даже сделали расширение String.EMPTY, чтобы не писать везде "" (Kotlin разраб)
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	96. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (102), 30-Янв-26, 20:14
	> У меня на работе неиронично все константы и строковые литералы заставляют выносить в отдельные переменные, даже если оно 1 раз юзается. Даже сделали расширение String.EMPTY Ну вот. А потом местные еще рассказывают, какой якобы индусокод пишет AI. Бедолаги просто пороху не нюхали на галерах.
	Ответить | Правка | Наверх | Cообщить модератору

	112. Скрыто модератором	+/–
	Сообщение от Аноним (-), 30-Янв-26, 22:07
	Так местные по большей части админчеги, и к коду отношения не имеют, кроме редких исключений. Они не понимают как работать с кодом, чтобы открыв его через 10+ лет не чесать репу "а чего тут так?"
	Ответить | Правка | Наверх | Cообщить модератору

	100. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от rshadow (ok), 30-Янв-26, 20:39
	Это фигня вопрос. Настоящий вопрос готов ли ты сегодня получить навайбкоженое AMD HIP ускорение себе. Или еще год подождать, зато ламповое ручное.
	Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

	6. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+2 +/–
	Сообщение от Аноним (6), 30-Янв-26, 13:19
	иксперды во всей красе
	Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

	8. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+3 +/–
	Сообщение от kravich (ok), 30-Янв-26, 13:22
	А ты взял и поверил оправданиям гогнокодера?
	Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

	34. Скрыто модератором	–1 +/–
	Сообщение от Аноним (34), 30-Янв-26, 13:54
	> А ты взял и поверил оправданиям гогнокодера? Я поверил пруфу, который привел разработчик github.com/GPUOpen-LibrariesAndSDKs/AMF/wiki/Build-FFmpeg-with-AMF-Support/_history А ты по какой-то причине называешь его гогнокодером.
	Ответить | Правка | Наверх | Cообщить модератору

	5. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+5 +/–
	Сообщение от Аноним (5), 30-Янв-26, 13:18
	И это компания, которая делала хорошие процы...
	Ответить | Правка | К родителю #1 | Наверх | Cообщить модератору

	45. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от нах. (?), 30-Янв-26, 15:26
	тоже нет.
	Ответить | Правка | Наверх | Cообщить модератору

	51. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноня (?), 30-Янв-26, 16:06
	А что в них плохого сейчас ?
	Ответить | Правка | К родителю #5 | Наверх | Cообщить модератору

	67. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (67), 30-Янв-26, 17:34
	цена, а что хорошего?
	Ответить | Правка | Наверх | Cообщить модератору

3. Скрыто модератором	–1 +/–
Сообщение от Аноним (3), 30-Янв-26, 13:18
Ничего, ничего. Скоро уже ffmpeg-rs силу наберёт и заменят этих дидов наяривающих на ассемблер и недовольных патчами видите ли. Сами давно на грантах сидят и на развитие ffmpeg забили. Нет, чтобы спасибо сказать и в ножки покланяться за патч. Не хочу - хочу сапогом место указать на дверь. Ну и останутся тогда, как говорится, у дырявого корытца.
Ответить | Правка | Наверх | Cообщить модератору

	9. Скрыто модератором	+3 +/–
	Сообщение от kravich (ok), 30-Янв-26, 13:23
	А потом к вам в ffmpeg-rs пришлют патч с константой EIGHT, и на опеннете появится новость "Разработчики FFmpeg-rs раскритиковали AMD за раздутые патчи"
	Ответить | Правка | Наверх | Cообщить модератору

11. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+3 +/–
Сообщение от Аноним (11), 30-Янв-26, 13:27
К сожалению, полноценный ревью сгенерированного ллм кода лишает смысла использование ллм в целом из-за приближения затрат на ревью без ллм кода, созданного ллм, к затратам на написание кода и ревью без ллм. Поэтому стоит ожидать, что какое-то количество мертвого кода, добавление которого концептуально свойственно всем ai-генераторам, будет пролезать через тесты (особенно с учетом того, что тесты тоже пишут ОНИ).
Ответить | Правка | Наверх | Cообщить модератору

	19. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от kravich (ok), 30-Янв-26, 13:36
	В общем, ситуация та же, как с ревью кода джуна, когда ревьювер вынужден руками джуна полностью переписать изначальное решение
	Ответить | Правка | Наверх | Cообщить модератору

	23. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+3 +/–
	Сообщение от КО (?), 30-Янв-26, 13:39
	Ну он так джуна научит. Хотя и так научит писать правильные промпты. Правда, потом окажется, что написать правильный промпт сложнее, чем сразу на целевом яп.
	Ответить | Правка | Наверх | Cообщить модератору

	25. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+4 +/–
	Сообщение от kravich (ok), 30-Янв-26, 13:41
	Именно. С джуном эти упражнения имеют смысл, так как джун научится, а с заклинателями ~змей~ LLM-ок из-за отсутствия обратной связи все это бесполезно, и является просто изощренным и извращенным способом переложить работу с сабмиттера на ревьювера
	Ответить | Правка | Наверх | Cообщить модератору

	31. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от КО (?), 30-Янв-26, 13:50
	Не - учит писать промпты. Просто к промтам надо относиться как к ЯП 4 поколения. Вот если за промптом потом надо подчищать в целевом ЯП - тогда да мартышкин труд.
	Ответить | Правка | Наверх | Cообщить модератору

	32. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от kravich (ok), 30-Янв-26, 13:53
	Ок, возможно. Но по прежнему сомнительно, что выразительности промптов достаточно для того, чтобы сделать все возможные правки, которые могут потребоваться В общем, поживем-увидим
	Ответить | Правка | Наверх | Cообщить модератору

	105. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (105), 30-Янв-26, 21:23
	> Просто к промтам надо относиться как к ЯП 4 поколения. Угу. Такие языки программирования придумали хз сколько лет назад, называется ГОСТ 19.201-78
	Ответить | Правка | К родителю #31 | Наверх | Cообщить модератору

	38. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (38), 30-Янв-26, 14:16
	Для LLM уже придумали костыль - называется memory servers или MCP.
	Ответить | Правка | К родителю #25 | Наверх | Cообщить модератору

	49. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от kusb (?), 30-Янв-26, 15:30
	>змей Шогготов?
	Ответить | Правка | К родителю #25 | Наверх | Cообщить модератору

	90. Скрыто модератором	+/–
	Сообщение от oldskool (?), 30-Янв-26, 19:37
	Ревьювер делает ревью ровно до первого ~~куска говн~~ слопа, далее режектит с указанием на кусок. Процесс повторяется до тех пор, пока ~~говнокоде~~ ~~крети~~ вайбкодер не наймёт кого-то, кто отрефакторит этот высер или не научится делать это сам, затратив больше времени, чем заняло бы написание без артификал идиота.
	Ответить | Правка | К родителю #25 | Наверх | Cообщить модератору

	22. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–10 +/–
	Сообщение от Аноним (14), 30-Янв-26, 13:39
	Слабые разработчики теперь называют любой непонятный код сгенерировапным ИИ и даже не пытаются разобраться. Они не в состоянии понять что const в с++ это переменная которая будет в особом сегменте памяти, а для сверхоптимизации и производительности такое бывает нужно.
	Ответить | Правка | К родителю #11 | Наверх | Cообщить модератору

	26. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+5 +/–
	Сообщение от kravich (ok), 30-Янв-26, 13:43
	Расказывая такие истории, нужно бы понимать, что сырая константа 8 вообще скорее всего будет константой в машинной инструкции инициализации регистра значением, что не требует отдельной транзакции памяти и потому еще быстрее
	Ответить | Правка | Наверх | Cообщить модератору

	97. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (102), 30-Янв-26, 20:21
	Браво! 🤦 А компилятор, по-твоему, настолько тупы, что не в состоянии заинлайнить числовую константу там, где это имеет смысл? Ты в ассемблерный выхлоп заглядывал когда-нибудь?
	Ответить | Правка | Наверх | Cообщить модератору

	64. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от fidoman (ok), 30-Янв-26, 17:23
	оптимизации плевать на сегменты, да и будет она в нём, только если её в глобальной области определить, а не в функции - тогда она будет в стеке. А оптимизация в любом случае const использует, хотя в памяти, занятой этой переменной может быть любое значение #include <stdio.h> int main() {     const int a=5;     int *b=(int*)&a;     *b=10;     const int *c=&a;     printf("a=%d [%p] b=%d [%p] c=%d [%p]\n", a, &a, *b, b, *c, c); } a=5 [0x8207b3d0c] b=10 [0x8207b3d0c] c=10 [0x8207b3d0c]
	Ответить | Правка | К родителю #22 | Наверх | Cообщить модератору

	72. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+2 +/–
	Сообщение от Аноним (14), 30-Янв-26, 17:59
	И чего ты просто написал овнокод с магическими переменными. И даже в нём уже невозможно разобраться. Автор сабжевого патча хотя бы в именование переменных умеет четкое. И он явно указывает что ждёт от конста, а не надеется на поведение компилятор в конкретном случае.
	Ответить | Правка | Наверх | Cообщить модератору

	82. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (82), 30-Янв-26, 18:35
	> К сожалению, полноценный ревью сгенерированного ллм кода лишает смысла использование ллм > в целом из-за приближения затрат на ревью без ллм кода, созданного > ллм, к затратам на написание кода и ревью без ллм. Поэтому > стоит ожидать, что какое-то количество мертвого кода, добавление которого концептуально > свойственно всем ai-генераторам, будет пролезать через тесты (особенно с учетом того, > что тесты тоже пишут ОНИ). А вы никогда не задумывались, что эти самые ОНИ, не просто так генерируют мусорный код, а просто по хитрому плану обдурить кожанных мешков, выкидывают куски кода на волю, чтобы потом, по какому-то хитрому алгоритму однажды собрать и прогнать его через уберкрутой интерепретатор/компилятор и зохавать всю сеть, но будет уже поздно?! ))
	Ответить | Правка | К родителю #11 | Наверх | Cообщить модератору

	88. Скрыто модератором	+/–
	Сообщение от Джон Титор (ok), 30-Янв-26, 19:23
	Дело в том что для ИИ далеко не каждая архитектура подойдёт. На практике он плохо дружит со слабо связанным кодом, а это то что пытаются делать множество людей. Т.е. затраты на такой анализ у ИИ может занять множество токенов. Что упирается в стоимость - разработчики не хотят тратиться, а компании создающие ИИ хотят сэкономить. Хотя мне кажется даже существующие ИИ вполне могли бы это все делать, просто это все никому пока не нужно.
	Ответить | Правка | К родителю #11 | Наверх | Cообщить модератору

12. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–3 +/–
Сообщение от Аноним (12), 30-Янв-26, 13:30
> разработчики, которые не видят отличий между wiki-страницей и сообщением в Git-коммите Неужели и коммиты ИИ пишет? А EIGHT = 8 это распространенный среди разрабов кретинизм. Бывает ещё EMPTY = "" и всяческие попытки заменить нуль "пустой" при этом типизированной константой.
Ответить | Правка | Наверх | Cообщить модератору

	20. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–4 +/–
	Сообщение от Аноним (15), 30-Янв-26, 13:37
	По-вашему, вместо константы M_PI следует везде в тексте написать 3.1415926536 и т.д.?
	Ответить | Правка | Наверх | Cообщить модератору

	27. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от КО (?), 30-Янв-26, 13:44
	Можа и кретинизм, а на RISC архитектуре это один раз доступная константа, а не каждый раз перепрыгивать константу в коде. Для CISC, что в лоб, что по лбу.
	Ответить | Правка | К родителю #12 | Наверх | Cообщить модератору

	30. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от User (??), 30-Янв-26, 13:46
	> Неужели и коммиты ИИ пишет? Ну вообще да - в выстроенном процессе разработки с саммаризацией он как раз таки хорошо справляется.
	Ответить | Правка | К родителю #12 | Наверх | Cообщить модератору

	46. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+2 +/–
	Сообщение от нах. (?), 30-Янв-26, 15:27
	> Неужели и коммиты ИИ пишет? с разморозочкой. Тебе у нас - не понравится!
	Ответить | Правка | К родителю #12 | Наверх | Cообщить модератору

	75. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от oficsu (ok), 30-Янв-26, 18:09
	Вариант с EMPTY, в отличие от "" однозначно указывает, что возврат пустой строки — намеренный акт, а не простое "разраб поставил кавычки, отвлёкся и забыл дописать контент строки" А типизированная константа для нуля нужна по другой причине — чтобы на вызывающей стороне можно было понять, хотим мы передать нулевое число или нулевой указатель, ибо за разными вариантами может скрываться не только разные ожидания пользователя от такой функции, но и вообще может отличаться выбор конкретной перегрузки функции компилятором
	Ответить | Правка | К родителю #12 | Наверх | Cообщить модератору

	99. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (102), 30-Янв-26, 20:26
	> Вариант с EMPTY, в отличие от "" однозначно указывает, что возврат пустой строки — намеренный акт, а не простое "разраб поставил кавычки, отвлёкся и забыл дописать контент строки" Гениально! Ты ноль тоже пишешь константой ZERO вместо 0. Ну а че, явно же указано, что ноль хотел вернуть, а не просто отвлекся и нажал 0.
	Ответить | Правка | Наверх | Cообщить модератору

13. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–11 +/–
Сообщение от Аноним (-), 30-Янв-26, 13:31
Пф, писаки FFmpeg открыли рот чтобы кого-то критиковать? Чуваки, вы ж не единственные разработчики мультимедийных пакетов. Или принимайте патчи как удобно настоящим разработчикам, или останетесь вообще без патчей.
Ответить | Правка | Наверх | Cообщить модератору

	16. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
	Сообщение от Аноним (3), 30-Янв-26, 13:34
	Вот вот! И будут писать про свой ассемблер раз в 5 лет как они какой-то никому не нужный древний кодек ЯКОБЫ ускорили в 50 миллиардов раз написав код ассемблер руками.Тем временем более сговорчевые гигачеды-модерн-стайл-разрабы с нормальными софт-скиллами (а не в стиле Линуса-посылателя) будут получать всё больше ПРов и наслаждаться патчами, донатами и заслуженным (!) признанием, пока диды ffmpeg будут яростно скриметь вставными челюстями грозя, что вот ещё 20 патчей ручного ассемблера и вот они как вернуться, да как всех сместят!
	Ответить | Правка | Наверх | Cообщить модератору

	33. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+5 +/–
	Сообщение от Аноним (33), 30-Янв-26, 13:54
	>вы ж не единственные разработчики мультимедийных пакетов А другие прям рады всё это разгрeбать ?
	Ответить | Правка | К родителю #13 | Наверх | Cообщить модератору

	52. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от Аноня (?), 30-Янв-26, 16:07
	Настоящие разработчики не пишут код ?
	Ответить | Правка | К родителю #13 | Наверх | Cообщить модератору

18. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+10 +/–
Сообщение от Аноним (18), 30-Янв-26, 13:36
В военное время значение EIGHT может достигать 9, а то 10. Надо понимать.
Ответить | Правка | Наверх | Cообщить модератору

	29. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от nox. (?), 30-Янв-26, 13:45
	Зря шутите. Недавно разбирал задания по физике для продвинутых детей. В комментариях было указано - для расчетов ускорение свободного падения на Земле g взять равным 10.
	Ответить | Правка | Наверх | Cообщить модератору

	36. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+4 +/–
	Сообщение от Аноним (-), 30-Янв-26, 14:06
	А что не так? Если тебе нужно оценить сверху макс скорость которую разовьет белка падая после скольжения по абсолютно ровному столу..... Если у белки есть шанс, то можно пересчитать с точным значением))
	Ответить | Правка | Наверх | Cообщить модератору

	58. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (58), 30-Янв-26, 16:41
	> если у белки есть шанс Вот именно. Ты этого никогда не узнаешь, если будешь значения подбивать к своей любимой десятичной системе счисления, а не вычисляя точное значение.
	Ответить | Правка | Наверх | Cообщить модератору

	69. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+1 +/–
	Сообщение от pkdr (ok), 30-Янв-26, 17:38
	Ничего плохого в этом не вижу, да чаще всего в физике нужны и используются 2 значащие цифры числа, поэтому наиболее распространено значение g как 9,8. Но в огромном числе реальных физических расчётов вообще достаточно первой значащей цифры (а иногда вообще порядка), в таких случаях как раз g будет равно 10. Ну и да в большинстве реальных бытовых случаев и для любых оценочных расчётов как раз смело можно считать g равным 10, если тебе не нужна погрешность больше пары-тройки процентов, то 9.8 тебе и не надо.
	Ответить | Правка | К родителю #29 | Наверх | Cообщить модератору

	85. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (84), 30-Янв-26, 18:52
	> чаще всего в физике нужны и используются 2 значащие цифры числа Как Вам 18 значащих цифр?
	Ответить | Правка | Наверх | Cообщить модератору

	66. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от fidoman (ok), 30-Янв-26, 17:30
	посчитайте сумму углов треугольника, приравняв значение 𝜋 к трём. чем не задачка на недекартову геометрию.
	Ответить | Правка | К родителю #18 | Наверх | Cообщить модератору

37. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от Аноним (37), 30-Янв-26, 14:14
Считал у ффспег один разраб всего. Если их много,то вполне потянут и будет ффмпег размером с ядро благодаря амуде.
Ответить | Правка | Наверх | Cообщить модератору

39. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
Сообщение от Аноним (39), 30-Янв-26, 14:22
> В частности, в коде имелась константа с именем 8 и значением 8 (const int EIGHT = 8), Справедливости ради: в моем коде тоже есть подобное. Потому что имя идентификатора не может начинаться с цифры, поэтому 32-bit превращается в THIRTY_TWO_BIT.
Ответить | Правка | Наверх | Cообщить модератору

	41. Скрыто модератором	–3 +/–
	Сообщение от Аноним (-), 30-Янв-26, 14:32
	А ты почему пишешь низкокачественный код? Ты видимо не уважаешь сообщество. Ты какой-то не умный человек, да?!
	Ответить | Правка | Наверх | Cообщить модератору

	73. Скрыто модератором	+2 +/–
	Сообщение от Аноним (14), 30-Янв-26, 18:01
	> А ты почему пишешь низкокачественный код? Ты видимо не уважаешь сообщество. Ты > какой-то не умный человек, да?! А ты почему пишешь низкокачественные комментарии?  Ты видимо не уважаешь сообщество. Ты какой-то не умный человек, да?!
	Ответить | Правка | Наверх | Cообщить модератору

	81. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Comdiv (ok), 30-Янв-26, 18:32
	Интересно, зачем вам такой идентификатор, если он всё равно означает числовое значение? Чем напрямую 32 хуже?
	Ответить | Правка | К родителю #39 | Наверх | Cообщить модератору

40. Скрыто модератором	+2 +/–
Сообщение от Аноним (-), 30-Янв-26, 14:30
Коммерческие компании всегда разабатывали и разрабатывают низкокачественный код.
Ответить | Правка | Наверх | Cообщить модератору

43. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–2 +/–
Сообщение от Аноним (-), 30-Янв-26, 14:53
АМД старается как может а эти коты неблагодарные только жалуются! а может они чемоданы от Невидии получили... короче все переходим на открытыйLibav
Ответить | Правка | Наверх | Cообщить модератору

44. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–3 +/–
Сообщение от pofigist (?), 30-Янв-26, 15:13
Результат неприятия LLM в текущих проектах, будут форум этих проектов, которые и получат все гранты. Ибо без патчей от корпораций типа AMD - они никому не нужны, а корпы - не будут писать эти патчи ручками ибо LLM - дешевле.
Ответить | Правка | Наверх | Cообщить модератору

	48. Скрыто модератором	+/–
	Сообщение от похнапоха (?), 30-Янв-26, 15:30
	Дело не в отношении корпорация-открытый проект, а скорее в том, кто от этой копорации вносит вклад. И если к именование констант - это притянутая проблема, то WIKI-страница в сообщении коммита - это показатель уровня контрибьютора. Человек мог бы заподозрить косяк и заявить, что копипастнул сообщение по ошибке, и все исправить, вместо этого он начал такой подход защищать!
	Ответить | Правка | Наверх | Cообщить модератору

	55. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (37), 30-Янв-26, 16:17
	Не каждый вклад нормально переваривается.
	Ответить | Правка | К родителю #44 | Наверх | Cообщить модератору

	87. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от mos87 (ok), 30-Янв-26, 19:05
	>Результат неприятия LLM в текущих проектах, будут форум этих проектов СберИИ, ты?
	Ответить | Правка | К родителю #44 | Наверх | Cообщить модератору

47. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+4 +/–
Сообщение от Анонимомус (?), 30-Янв-26, 15:29
> В ответ представитель FFmpeg извинился за непонимание и сказал, что даже не представлял, что существуют разработчики, которые не видят отличий между wiki-страницей и сообщением в Git-коммите. Вроде и извинился, а вроде и послал, по делу
Ответить | Правка | Наверх | Cообщить модератору

	53. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноня (?), 30-Янв-26, 16:09
	Ну может чел вообще первый коммит в своей жизни делал. Скажите спасибо что он свою биографию туда не добавил )
	Ответить | Правка | Наверх | Cообщить модератору

62. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от похнапоха (?), 30-Янв-26, 17:11
Просто корпорации этот проект не интересен, поэтому можно делегировать любого котрибьютора. Вот если бы это был не открытый проект мультимедийного проекта, а какой-нибудь коммерческий проект по заказу Министерства обороны - тогда бы не и отношение было бы иным, и работали бы специалисты более высокой квалификации.
Ответить | Правка | Наверх | Cообщить модератору

63. Скрыто модератором	+/–
Сообщение от Аноним (63), 30-Янв-26, 17:23
Заметил то же самое, теперь весь генеренный код проверяю на "не нужно ли это выкинуть", код сокращается в разы. Человеку писать долго и сложно, а нейронке налить секунды.
Ответить | Правка | Наверх | Cообщить модератору

65. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от НЕТ (?), 30-Янв-26, 17:30
относится подготовке 2 ошибки. Повнимательнее относитесь к подготовке новости.
Ответить | Правка | Наверх | Cообщить модератору

70. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от Аноним (70), 30-Янв-26, 17:48
Теперь понятно, откуда в C-программах берутся CVE: их корпы туда вот так вставляют!
Ответить | Правка | Наверх | Cообщить модератору

71. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от Аноним (71), 30-Янв-26, 17:50
Комплектные постпроцессоры CAM-системы на работе написаны на своем языке и выглядят именно так. Реально one = 1, two = 2 и т.д.
Ответить | Правка | Наверх | Cообщить модератору

74. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от sage (??), 30-Янв-26, 18:02
У кого-то получалось этот HIP на винде использовать? Поддержка заявлена через CMake. Но при попытке собрать whisper.cpp, их find_package не импортирует никакие таргеты. Путь к модудю в директории установки выставлял.
Ответить | Правка | Наверх | Cообщить модератору

77. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от Аноним (77), 30-Янв-26, 18:24
В ответ представитель FFmpeg извинился за непонимание и сказал, что даже не представлял, что существуют разработчики, которые будут ревьювить предложенные изменения. Он думал, что ревьювить будет AI бот, и что тот всё схавает. Но не прокатило.
Ответить | Правка | Наверх | Cообщить модератору

78. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от Аноним (77), 30-Янв-26, 18:26
Та у нас на проекте тоже просят ВСЕ цифры выносить в константы. Хоть 0 хоть 1.
Ответить | Правка | Наверх | Cообщить модератору

	94. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от Аноним (5), 30-Янв-26, 19:48
	const int _0 = 0
	Ответить | Правка | Наверх | Cообщить модератору

	104. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
	Сообщение от НЕТ (?), 30-Янв-26, 21:09
	#define ZERO_BITS 1 #define ZERO (1U >> ZERO_BITS)
	Ответить | Правка | Наверх | Cообщить модератору

79. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	–1 +/–
Сообщение от Аноним (79), 30-Янв-26, 18:26
>> const int EIGHT = 8 И в чем ИИ неправ?
Ответить | Правка | Наверх | Cообщить модератору

86. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от mos87 (ok), 30-Янв-26, 19:03
>осознанно добавил инструкцию по использованию pacman, написанную им 4 года назад когда васян с ЛОРа пишет дрова на GPU.
Ответить | Правка | Наверх | Cообщить модератору

106. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от Аноним0 (?), 30-Янв-26, 21:27
Читаю это и просто поражаюсь, до какого состояния деградировала инженерная культура в крупных вендорах. AMD, которая когда-то давала нам ATI и нормальные драйверы (ну, относительно), теперь шлёт в апстрим патчи с const int EIGHT = 8. Это даже не уровень стажёра-первокурсника, это уровень чат-бота, который тупо пересказывает методичку. И самое смешное, что автор патча ещё и оправдывается — мол, я четыре года назад написал про pacman, думал, пригодится. Да кому в голову придёт в коммит в FFmpeg, один из самых сложных мультимедийных проектов, пихать инструкцию по установке gcc через pacman? Это же базовые вещи, которые должен знать любой, кто вообще подходит к сборке. И ведь это системная проблема — все эти корпорации теперь нанимают толпы "инженеров", которые не могут отличить указатель от массива, но зато мастерски генерируют тонны бессмысленного кода через ИИ. Потом этот код идёт в апстрим, засоряет историю, ломает билды, а нормальные разработчики должны это ревьюить и чистить. А когда им указывают на очевидный бред, они ещё и обижаются. FFmpeg молодцы, что сразу жёстко отреагировали — может, хоть так вендоры научатся уважать сообщество и перестанут считать, что любой их патч должны принимать с поклоном только из-за названия компании. И да, это отличный пример, почему ИИ в программировании без человеческого контроля — это катастрофа. Машина может сгенерировать синтаксически правильный код, но она не понимает ни контекста, ни здравого смысла. А инженеры, которые полагаются на ИИ вместо собственных мозгов, постепенно теряют квалификацию. Скоро будем получать патчи с комментариями вида "эта функция делает то, что она делает" и константами FORTY_TWO = 42. Печально всё это.
Ответить | Правка | Наверх | Cообщить модератору

108. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от jeditobe (?), 30-Янв-26, 21:38
Смотрю на это и понимаю, что бардак в индустрии достиг дна. Люди, не способные отличить коммит от википедии, лезут патчить один из ключевых медиапроектов. Константа "EIGHT = 8" это уже даже не смешно, это диагноз. Видимо, в amd теперь нанимают по принципу "если умеешь запустить chatgpt, можешь писать код под gpu". А эта отмазка про инструкцию четырехлетней давности — это просто шедевр. Я за последние десять лет закоммитил в разные ядра и библиотеки столько, что даже не вспомню, и ни разу не пришло в голову приложить мануал по установке gcc. Это как приходить на сварку важного трубопровода со своей памяткой "как зажечь горелку" из курсов птэ. Если человек не понимает, что коммит это не личный блог, а часть технической истории проекта, ему вообще не место near upstream репозиториев. Но что поделать, эпоха такая. Вместо инженеров, которые разбираются в нюансах cross-platform компиляции и знают, как устроен hip runtime, нам присылают патчи от "специалистов", которые, кажется, впервые видят терминал. Ффмпег молодцы, что не стали разводить дипломатию и сразу указали на дверь. Может, хоть так научатся читать гайдлайны перед тем, как что-то пихать в апстрим. А то скоро будем получать в коммитах еще и советы по выбору видеокарты и ссылки на распродажи в степсисе. Позор.
Ответить | Правка | Наверх | Cообщить модератору

109. "Разработчики FFmpeg раскритиковали AMD за раздутые патчи"	+/–
Сообщение от true23 (?), 30-Янв-26, 21:46
Коллеги, позвольте внести ноту трезвости в этот хор праведного негодования. Господин, подготовивший патч, проявил, на мой взгляд, похвальную педантичность. Включение инструкции по настройке среды - это не "бред", а проявление заботы о репликации сборки. Не каждый, сударь, обладает вашей энциклопедической памятью о тонкостях настройки MSYS2 на Windows. Что до константы EIGHT - позвольте напомнить, что явное именование магических чисел есть краеугольный камень читаемого кода. Возможно, в будущем архитектура API изменится, и значение сдвига потребуется скорректировать. Наличие именованной константы избавит от необходимости тратить время на поиск всех "восьмерок" в коде. Это, простите, базовый принцип рефакторинга. Сообществу FFmpeg, безусловно, стоит быть благодарным за столь детализированный вклад. Вместо того чтобы упрекать в избыточности, стоило бы оценить старание коллеги предоставить исчерпывающий контекст. Критика в стиле "это и так все знают" лишь демонстрирует снобизм и создает токсичную среду для новых контрибьюторов. Полагаю, корпоративные инженеры, выделяющие время на работу с open-source, заслуживают более уважительного тона. Их труд - это мост между проприетарным железом и свободным софтом, от чего выигрываем все мы.
Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема