Неужели и коммиты ИИ пишет? А EIGHT = 8 это распространенный среди разрабов кретинизм. Бывает ещё EMPTY = "" и всяческие попытки заменить нуль "пустой" при этом типизированной константой.

Справедливости ради: в моем коде тоже есть подобное. Потому что имя идентификатора не может начинаться с цифры, поэтому 32-bit превращается в THIRTY_TWO_BIT.

Вроде и извинился, а вроде и послал, по делу

Но не прокатило.

И в чем ИИ неправ?

И ведь это системная проблема — все эти корпорации теперь нанимают толпы "инженеров", которые не могут отличить указатель от массива, но зато мастерски генерируют тонны бессмысленного кода через ИИ. Потом этот код идёт в апстрим, засоряет историю, ломает билды, а нормальные разработчики должны это ревьюить и чистить. А когда им указывают на очевидный бред, они ещё и обижаются. FFmpeg молодцы, что сразу жёстко отреагировали — может, хоть так вендоры научатся уважать сообщество и перестанут считать, что любой их патч должны принимать с поклоном только из-за названия компании.

И да, это отличный пример, почему ИИ в программировании без человеческого контроля — это катастрофа. Машина может сгенерировать синтаксически правильный код, но она не понимает ни контекста, ни здравого смысла. А инженеры, которые полагаются на ИИ вместо собственных мозгов, постепенно теряют квалификацию. Скоро будем получать патчи с комментариями вида "эта функция делает то, что она делает" и константами FORTY_TWO = 42. Печально всё это.

А эта отмазка про инструкцию четырехлетней давности — это просто шедевр. Я за последние десять лет закоммитил в разные ядра и библиотеки столько, что даже не вспомню, и ни разу не пришло в голову приложить мануал по установке gcc. Это как приходить на сварку важного трубопровода со своей памяткой "как зажечь горелку" из курсов птэ. Если человек не понимает, что коммит это не личный блог, а часть технической истории проекта, ему вообще не место near upstream репозиториев.

Но что поделать, эпоха такая. Вместо инженеров, которые разбираются в нюансах cross-platform компиляции и знают, как устроен hip runtime, нам присылают патчи от "специалистов", которые, кажется, впервые видят терминал. Ффмпег молодцы, что не стали разводить дипломатию и сразу указали на дверь. Может, хоть так научатся читать гайдлайны перед тем, как что-то пихать в апстрим. А то скоро будем получать в коммитах еще и советы по выбору видеокарты и ссылки на распродажи в степсисе. Позор.

Что до константы EIGHT - позвольте напомнить, что явное именование магических чисел есть краеугольный камень читаемого кода. Возможно, в будущем архитектура API изменится, и значение сдвига потребуется скорректировать. Наличие именованной константы избавит от необходимости тратить время на поиск всех "восьмерок" в коде. Это, простите, базовый принцип рефакторинга.

Сообществу FFmpeg, безусловно, стоит быть благодарным за столь детализированный вклад. Вместо того чтобы упрекать в избыточности, стоило бы оценить старание коллеги предоставить исчерпывающий контекст. Критика в стиле "это и так все знают" лишь демонстрирует снобизм и создает токсичную среду для новых контрибьюторов.

Полагаю, корпоративные инженеры, выделяющие время на работу с open-source, заслуживают более уважительного тона. Их труд - это мост между проприетарным железом и свободным софтом, от чего выигрываем все мы.

чего шуметь, чел просто хотел сказать "im use arch btw"

> const int EIGHT = 8

Откуда вы это взяли? Этого нет ни в обсуждении пулреквеста, ни ни в одном коммите который в этот ПР когда-либо пушили.

Там есть один комментарий от чела, который по-английски писать не умеет про "option const named 8 with value 8", но этот чел не только писать не умеет, он ещё и выделять код который комментирует тоже не умеет.

Отсюда вопрос - это вы галлюцинируете похуже старого чатагпт, или интернет тщательно подтёр все следы этой константы из нашего пространственно-временного континуума?

Вы не понимаете! Это BTW I Use Arch (c)