Форматирование в Rust

А если кратко, чем недоволен интурист то?

Линус при разрешении конфликта в Rust оформил импорты по одному на строку, чтобы избежать будущих конфликтов. Но rustfmt предложил объединить их в одну строку, что Линус назвал «bass-ackwards garbage» и раскритиковал автоматическое форматирование за усложнение работы.

Уж лучше пусть оно само форматирует, чем выслушивать на ревью, что у тебя импорты не в алфавитном порядке.

А дальше? Там больше текста.

А дальше сам.

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

Разработка ядра – это для Линуса уже чужой монастырь? Сильно!

Вот когда гит научится в language-based дифы, вот тогда и не понадобится импорты форматировать.

UPD: пизжу, оказывается у кого-то просто rustfmt не настроен.

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

Рекомендации по форматированию языка - чужой монастырь. Не хочешь пользоватья форматированием официальным, не пользуйся, форматируй сам

Раст сейчас продвигают те, чья рука уже даже не по локоть, а по плечо в жопе у Линуса. Так что врядли он станет сильно что то возражать. Раст сейчас это мейнстрим и он с явным недовольством это примет.

Не хочешь пользоватья форматированием официальным, не пользуйся, форматируй сам

То есть топик ты не читал.

Раст сейчас продвигают те, чья рука уже даже не по локоть, а по плечо в жопе у Линуса.

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

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

В этой стране никто не верит в то, что делает власть, вот и проецируют на разработку)

Раст сейчас это мейнстрим

А что такое мейнстрим? Судя по недавнему на ЛОРе выкладыванию всяких рейтингов ЯП, раст в отстое.

I noticed that people added multiple

  use crate::xyz;

next to each other, so I turned them into

  use crate::{
     xyz,
    abc,
  };

instead to make it easy to just add another crate without messing crap
up. The use statements around it had that format too, so it all seemed
sensible and visually consistent.

But then I run rustfmtcheck, and that thing is all bass-ackwards
garbage. Instead of making it clean and clear to add new rules, it
suggests

  use crate::{xyz, abc};

but I have no idea what the heuristics for when to use multiple lines
and when to use that compressed format are.

This is just ANNOYING. It's automated tooling that is literally making
bad decisions for the maintainability. This is the kind of thing that
makes future conflicts harder for me to deal with.

Miguel, I know you asked me to run rustfmtcheck, but that thing is
just WRONG. It may be right "in the moment", but it is

 (a) really annoying when merging and not knowing what the heck the rules are

 (b) it's bad long term when you don't have clean lists of "add one
line for a new use"

Is there some *sane* solution to this? Because I left my resolution
alone and ignored the horrible rustfmtcheck results.

I tried to google the rust format rules, and apparently it's this:

    https://doc.rust-lang.org/style-guide/index.html#small-items

can we please fix up whatever random heuristics? That small items
thing may make sense when we're talking things that really are one
common data structure, but the "use" directive is literally about
*independent* things that get used, and smushing them all together
seems entirely wrong.

I realize that a number of users seem to just leave the repeated

   use kernel::xyz;
   use kernel::abc;

as separate lines, possibly *becasue* of this horrendous rustfmt
random heuristic behavior.

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

Для мержей чуть лучше:

   use kernel::xyz;
   use kernel::abc;

чем:

  use crate::{
     xyz,
    abc,
  };

Хотя бесит эта портянка. И если добавят такой вариант в rustfmt, то будет очень хорошо.

С другой стороны в ядре хоть какой-то подход к этому есть вообще? А то rust/kernel/error.rs:

// SPDX-License-Identifier: GPL-2.0

//! Kernel errors.
//!
//! C header: [`include/uapi/asm-generic/errno-base.h`](srctree/include/uapi/asm-generic/errno-base.h)

use crate::{
    alloc::{layout::LayoutError, AllocError},
    str::CStr,
};

use core::fmt;
use core::num::NonZeroI32;
use core::num::TryFromIntError;
use core::str::Utf8Error;

З.Ы.: Интересно, а до глобального clang-format дойдёт?

Да кого волнует мнение какого-то там финского студента? Написал нам линукс — и свободен! (%

Про https://ru.wikipedia.org/wiki/Edlin он хорошо потроллил.

Edlin

А ничего, что ed/ex включены в любую полноценную UNIX-like операционную систему? (%

Тут попахивает откровенным сектантством…

Посмотрел, что тут пытался подправить Линус.

drivers/gpu/drm/nova/gem.rs:

use kernel::{
    drm,
    drm::{gem, gem::BaseObject},
    prelude::*,
    sync::aref::ARef,
};

drivers/gpu/drm/nova/file.rs:

use kernel::{
    alloc::flags::*,
    drm::{self, gem::BaseObject},
    pci,
    prelude::*,
    uapi,
};

drivers/gpu/drm/nova/driver.rs:

use kernel::{
    auxiliary, c_str, device::Core, drm, drm::gem, drm::ioctl, prelude::*, sync::aref::ARef,
};

Кто в лес, кто по дрова…

    pub(crate) fn new(dev: &NovaDevice, size: usize) -> Result<ARef<gem::Object<Self>>> {
        let aligned_size = size.next_multiple_of(1 << 12);

        if size == 0 || size > aligned_size {
            return Err(EINVAL);
        }

        gem::Object::new(dev, aligned_size)
    }

мда… may be panic

автоматизированного инструмента форматирования

Киньте ссылку на описание этого языка в формате БНФ.

То, что пропихивают корпорации, продукты которых являются основой всего сейчас. Не сам он из всех утюгов лезет.

Могу поспорить на самого жирного своего козленка, что ничего Линус не сделает расту. Максимум по выпендривается и не более. Ибо чревато.

А кто их читает? Ты недавно на ресурсе, да?

Киньте ссылку на описание этого языка в формате БНФ.

Ну ты захотел…

Современный сахарок такой затейливый…

Разработка ядра – это для Линуса уже чужой монастырь? Сильно!

Старикашку давно пора отменить за его неподобающее поведение, которое он демонстрировал 20 лет назад.
Геть его! Ату!
:D

Это не реально. И нужен будет уже не гит

Про https://ru.wikipedia.org/wiki/Edlin он хорошо потроллил.

Вспомнил молодость в Usenet?

А ничего, что ed/ex включены в любую полноценную UNIX-like операционную систему? (%

Много народу правит исходники в ed?

[ЖЖ]Полез в Википедию смотреть про ex, дочитал, машинально попытался закрыть вкладку при помощи :q [/ЖЖ]

https://doc.rust-lang.org/stable/reference/items/use-declarations.html

Не буквально БНФ, но близко.

а в ядро его взяли потому что он безопасный

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

Неужели в раст добавили return?

в ядро его взяли потому что он безопасный

Безопасный он только когда

Это была шутка юмора, основанная на заклинаниях растобоев.

Безопасный он только когда на нем пишут используя safe-семантику

нет, не только

большинство же разрабов используют unsafe что бы не воевать со статическим анализатором,

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

Неужели в раст добавили return?

с самого начала был

короче, нихрена не знаешь не понимаешь но мнение имеешь, перестань так делать

… и шутки у тебя дурацкие

zurg ★
07.10.25 10:38:40 MSK D’ Rust

Так он и был, только редко нужен.

большинство же разрабов используют unsafe что бы не воевать со статическим анализатором,

Странные слова. unsafe используют совсем по другим причинам.

https://doc.rust-lang.org/stable/reference/items/use-declarations.html
Не буквально БНФ

Это похоже на какой-то маленький мануал.

Т.е., формального описания синтаксиса языка, как было у Николауса Вирта, у них нет?

Т.е., формального описания синтаксиса языка, как было у Николауса Вирта, у них нет?

https://github.com/tree-sitter/tree-sitter-rust – но неофициальное.

Ну пусть напишут linux-rust-fmt, который будет форматировать под стиль, удобный товарищу Линусу. Делов-то.

нет, не только

Но называть эти самые «безопасные» фичи раста ты не станешь, потому что

большинство разрабов не воюют с компилятором а используют

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

Ансейф используют практически все кто пытается на нем реальные задачи решать, а не только бегать прославлять.

На моё ИМХО, ЛЮБОЕ автоматическое форматирование – зло. Т.к. всех ситуаций не предусмотришь, и во огромном количестве случаев оно просто сломает то, что с т.з. человека – удобно и эстетично. Что мы собственно и наблюдаем. Или, например, преобразует сами знаете в чёрт знает что такую довольно часто встречающуюся банальщину:

if (aaa  = bbb ) { ccc (ddd ); }
if (aaa2 = bbb2) { ccc2(ddd2); }
if (aaa3 = bbb3) { ccc3(ddd3); }

Вот будет эта тупая железяка код писать – пусть свой собственный код и форматирует как ей заблагорассудится. А в мой код лезть не надо.

Ещё не предлагали улучшить резолв конфликтов в гите?

Единообразие важнее хрупких чувств ко всякой ерунде.

Ну если для вас ваш собственный код – ерунда… то в общем верю. Равнодушных ремесленников – в любой профессии большинство.

Отвечу словами Линуса:

And quite frankly, for most of the real problems (as opposed to the stupid
bugs - of which there are many, as the latest crap with "truncate()" has
shown us) a debugger doesn't much help. And the real problems are what I
worry about. The rest is just details. It will get fixed eventually.

(Речь идёт о дебаггере, не форматтере. Но учитывая ещё большую отдалённость форматтера от «реальных проблем», смысл сообщения усиливается.)

https://web.archive.org/web/20250825062812/https://lkml.org/lkml/2000/9/6/65

6 Sep 2000

За 25 лет изменений не было?