LINUX.ORG.RU

Линус Торвальдс сказал «нет» коду с предупреждениями компиляции

 , , , ,


0

1

Линус Торвальдс изменил параметры компиляции кода ядра Linux по умолчанию. Теперь при сборке используется параметр -Werror, приводящий к интерпретации любого предупреждения как ошибки и, соответственно, к остановке сборки ядра с этой ошибкой. Данное изменение конфигурируется новым параметром WERROR, который включен по умолчанию.

По мнению Линуса, изменение должно повлиять на качество кода и в частности, на качество пулл-реквестов, которые он получает. Для большего эффекта он постучал по дереву (Knock wood) прямо в сообщении к коммиту.

>>> Подробности



Проверено: Shaman007 ()
Последнее исправление: hobbit (всего исправлений: 3)

Это хорошо и правильно, вот только появление ворнингов на одном и том же коде зависит от версии компилятора, разве нет? А где гарантия, что ворнингов не будет ни на одной версии GCC?

Xintrea ★★★★★
()

Еще бы анализатором гонять.
Хорошо было бы если бы какой-нибудь clang-analyzier туда пришпендюрили в автоматическом режиме и не давали компилить если предупреждения появляются.

PPP328 ★★★★★
()
Ответ на: комментарий от B0B

Как раз роллинги это врядли затронит. Другое дело всякие кастомные сборки со своими патчами.

hummer
() автор топика

Постучать по дереву - вот решение технологических задач в 21 году.

Аплодирую стоя!

Зы. Чем-то мне это наши реалии напоминает. Вопрос как бы не в деньгах вовсе. Дерево!

anonymous
()

стоящая новость. линус - Ъ.

crypt ★★★★★
()
Ответ на: комментарий от anonymous

Постучать по дереву - вот решение технологических задач в 21 году.

Вспомнился начальник отдела, одной крупной компании, в которой я очень давно работал. Перед одним важным релизом он развесил связки чеснока на дверных ручках. Впрочем это не помогло и релиз был проблемным.

hummer
() автор топика
Ответ на: комментарий от BceM_IIpuBeT

Сомневаюсь, что она интегрируется с Uemacs, которой пользуется Линус.

hummer
() автор топика
Ответ на: комментарий от anonymous

Так то пляски непосредственно тех, кто что-то делает сам, а тот начальник отдела лишь руководил. Впрочем прикольный был мужик. На собеседовании задал самый прикольный вопрос за мою карьеру, про выбор красивой девушки среди стоящих вдоль дороги, без возможности вернуться назад.

hummer
() автор топика
Ответ на: комментарий от hummer

Так то пляски непосредственно тех, кто что-то делает сам, а тот начальник отдела лишь руководил

По логике его задача обеспечить бубнами и остальным стаффом…

про выбор красивой девушки среди стоящих вдоль дороги, без возможности вернуться назад.

Судя по всему тест прошли удачно.

anonymous
()
Ответ на: комментарий от anonymous

По логике его задача обеспечить бубнами и остальным стаффом…

В итоге там всё решили как раз кодоанализатором, который выявил все глупые ошибки в коде на C/C++.

Судя по всему тест прошли удачно.

Да, предложил первую половину пути никого не выбирать, а собирать статистику и лишь затем выбрать, когда попадётся кто-то лучше среднего показателя.

hummer
() автор топика
Последнее исправление: hummer (всего исправлений: 1)
Ответ на: комментарий от hummer

Проще ведь в конфигурации выключить.

Локальная конфигурация - это не патч в единственно верный репозитарий распределенной системы.

anonymous
()
Ответ на: комментарий от hummer

Да, предложил первую половину пути никого не выбирать, а собирать статистику и лишь затем решить, когда попадётся кто-то лучше среднего показателя.

А 5сли распределение девушек не равномерно, возьмёшь самую последнюю какая есть?

anonymous
()
Ответ на: комментарий от anonymous

В том-то и дело, что всех нельзя - назад дороги нет, только вперёд. Увидел красавицу, а дальше может быть ещё красивее, но может и не быть. Мой ответ конечно же не универсальный, но ему понравился.

hummer
() автор топика
Ответ на: комментарий от hummer

Ваш ответ - это и есть (как можно больше помотреть), что совершенно естественно природе мужчины. А дай волю и не только посмотреть…:)

anonymous
()

Только за двумя руками, двумя ногами, двумя ушами, ну и ещё один голос можно подсчитать - но озвучивать не буду 😉

drfaust ★★★★★
()
Ответ на: комментарий от B0B

А причём тут роллинг-дистры - он для ванильного ядра это сказал - а дальше, да хоть свои бэкдуры от майоров как с нашей, так и с другой стороны вставляй…

drfaust ★★★★★
()
Ответ на: комментарий от anonymous

Распределение априори неравномерно. Хотя бы потому, что у каждого свои предпочтения…

anonymous
()

Там всегда серьёзно относились к предупреждениям, давно пора.

anonymous
()
Ответ на: комментарий от intelfx

Ну ты сравнил. Эта новость очень по теме.

А дальше всё верно подмечено. Правда нао добавить, что Линуса купили или сделали «ай-ай-ай». Я до сих пор случай с доцей вспоминаю.

B0B
()

Предлагаю переиначить слова «прощание славянки» под реалии линукс ядра, исполнить и отправить линусу ссылку. Ключевые слова: «тех кто предал тебя и продал».

anonymous
()
Ответ на: комментарий от B0B

Я до сих пор случай с доцей вспоминаю.

Не все настолько стойкие как Сталин.

BceM_IIpuBeT ★★☆☆☆
()
Ответ на: комментарий от intelfx

Думаю это заслуженное внимание к его персоне не конкретно из-за этой новости. А последовательности новостной ленты по теме.

anonymous
()
Ответ на: комментарий от intelfx

Тут ты прав. Но все мы люди. И театральность нужна.

B0B
()

Линус Торвальдс сказал «нет» коду с предупреждениями компиляции

Ну и молодец ::)

К слову, а я забыл или не помнил, как выделать секции где по сути предупреждение есть, но по логике быть не должно, типа

#pragma ERROR_TYPE_XXX_IGNORE_START

BLABLACODE

#pragma ERROR_TYPE_XXX_IGNORE_END

А то иногда ругань есть, но она необоснованна ибо у гцц либастрала нет почему я делаю так и тут всё хорошо.

LINUX-ORG-RU ★★★★★
()

Этот параметр нужен разрабам, пользователям от него только лишний гемор, а разрабы и так могли с ним компилять свой код

IvanR ★★★
()
Ответ на: комментарий от intelfx

Плюс в стуке по дереву есть великий символизм, говорящий о том, что он сам прекрасно отдает себе отчет в происходящем и уповает не на действие, а на «чудо».

anonymous
()
Ответ на: комментарий от anonymous

Понимая, что чуда не случится. Ибо он знает чуточку больше чем мы о том, какие шаги будут предприняты в обозримом будущем. Такие дела.

anonymous
()
Ответ на: комментарий от anonymous

Ну а изменение параметра по-умолчанию и не может что-то изменить. Его и раньше ручками можно было… Все это трагикомичный фарс, посоны.

anonymous
()
Ответ на: комментарий от Xintrea

А можно примеры? Желательно те, где это новые ворнинги возникли «на пустом месте», а не просто новая версия компилятора проверяет те косяки, которые не проверяла старая.

Не то чтобы я придираюсь, но на практике встречал только второе и это, блин, правильно.

RiseOfDeath ★★★★
()
Последнее исправление: RiseOfDeath (всего исправлений: 2)

ЕМНИП, как-то давно Торвальдс ругался, на тех, кто исправляет код, в ответ на ворнинги компилятора. Мол, не понимают что делает код, ворниг то исчезает, но исправленный код работает не корректно. А мантенеры должны внимательно изучать и отсеивать такие вредные патчи, ведь исходный код с ворнингом работал корректно...

Теперь он хочет, чтобы его завалили подобными патчами, особенно при смене версии gcc?

mky ★★★★★
()
Ответ на: комментарий от hummer

предложил первую половину пути

Не эксперт по теорверу и матстатистике, и всяким подсчетам доверительных интервалов и тп. Но…

Половина - слишком большая выборка для оценки среднего значения. Думаю, хватило бы первых несколько «красоток» для оценки средней «красоты» с достаточной погрешностью. Слишком большие потери

anonymous
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.