clang-format considered harmful

clang-format considered harmful

I don't know of a good C++ code formatter, but clang-format is not one. It is a thing I don't like about C++.

1. Myths

First, I must dispel 4 oft-cited myths used to argue for uncritically slapping on a random coding standard.

Myth 1: Any coding standard is as good as another.

There is such a thing as practicality in coding standards. Before even contemplating controversial topics – aesthetics, it's not hard to think of aspects of code formatting that contribute to write amplification – how big a change becomes in the resulting diff – that should be uncontroversial. Let's get the basics right:

  • Ease of adding or removing an element at the end. This thing:

    XXX XXX = XXX::XXXXXXXXXXXXXX { X,
                                    Y }
    
  • Reflow ripple effects: This thing, after inserting "something":

    XXX XXX = XXX::XXXXXXXXXXXXXX { "something",
                                    X, Y }
    
  • Dependent indentation: This thing, after you renamed it (and forgot to reformat):

    XXX XXX = XXX::XXXX { X,
                                    Y }
    
  • Indentation dependence: When changing the scope of a section of code causes the formatter to change strategy:

    {
        XXX
        XXX =
            XXX::
            XXXXX
            { X, Y }
    }
    
  • Human predictability: Does the formatter follow rules that a human with infinite experience can predict, or does it play chess with sums of weighted costs? In practical terms, must you run a formatter locally in order to write anything that CI will accept?

There exists a formatting that has none of these problems. I'm of course talking about the self-evident pythonic/rustic formatting (which probably has many more names):

XXX XXX = XXX::XXXXXXXXXXXXXX {
    X,
    Y,
}

Myth 2: The most important thing is to have a coding standard and enforce it.

I remember a time before clang-format: I would say that professional developers did at least as good of a job as clang-format to begin with. In fact, in some ways better than any autoformatter could ever come up with, because the human knows best, such as which arguments are associated. Freedom of expression! This openness to creativity made the conventions fluid, so that better ideas had a foothold.

In contrast, with clang-format, I see good developers being passive and indifferent to details like trailing comma that are not at all insignificant to what clang-format will do.

If the purpose of automatic formatting is to avoid style disputes in code review, it doesn't work, because too few people know the importance it gives to trailing comma – I have to nag people about it.

Myth 3: It is possible to configure clang-format to a pythonic/rustic style.

I have tried every config option. There is AlignAfterOpenBracket, but you have to do the rest yourself in terms of remembering trailing comma (which is only applicable in curly braces and not enforceable), forcing line breaks with line comments and liberal use of // clang-format off.

Myth 4: It is always convenient for everyone to run the formatter.

If you haven't noticed the trend, everything is wrapped in impenetrable all-encompassing dockerized CI-scripts that can't just check a small change quickly.</sarcasm>

It doesn't actually matter how convenient it is, because I don't necessarily approve of what it does to my code – I can't run the formatter before I have committed my changes anyway. Then, I rewrite my code to comply if needed be. If revising one's commit stack isn't hard enough as it is, doing it with style changes into the mix is the worst.

2. Properties of a good formatter

  • Sensible by default, or a sensible configuration (after the criteria above) must exist in its configuration space.
  • Slack: The human knows certain things better than the formatter, such as which arguments are associated, and may therefore have a preference for where to break the line if necessary. The formatter's job is not to take this freedom of expression away. Its job is to satisfy a disjoint set of requirements. Therefore, it must allow more than one way to lay out the same code. Python formatters do a good job in this department.
  • Humanly predictable.

3. So what's wrong with clang-format in particular?

All the above. If clang-format behaved like a python formatter or like rustfmt, you wouldn't be reading this. Though I'm no fan of automatic formatting in general, other languages have it better.

Discussion 42 comments · 9 points · anordal · 2022-05-28
Open on Lobsters
Loading the discussion…

Domain filters

Stories from these domains are hidden from every list. Subdomains match too: blocking substack.com also hides danluu.substack.com.

    New collection

    Delete this collection?

    About YAVCHN

    YAVCHN is a reader for Hacker News and Lobsters, with articles and discussions in separate windows or Classic pages.

    Created by Paul Parks and built with PUDL.

    YAVCHN source code on GitHub

    Help

    Keyboard

    j / k
    Move down and up the story list. The arrow keys scroll whatever has focus.
    Enter
    Read the marked story in the article reader.
    ]
    Read the next story in the same article-reader applet. Back returns to the previous story.
    p
    Pin or unpin the marked story, which keeps it in Pinned.
    n / N
    Move to the next or previous top-level comment in the window in front.
    c
    Collapse or expand that comment.
    f
    Hide or show the story list.
    Esc
    Close a menu or this help.
    Access key m
    Go to the menu bar. Most browsers take it with Alt on Windows and Linux, and Safari with Control and Option.
    ?
    Show this help.

    Windows

    Each story opens in a window holding its article above its discussion; drag the bar between them to share the room differently. A window can be moved by its title bar, resized from any edge, snapped to a half or a corner by dragging it there, maximised, or minimised to the bar at the foot of the page. Use Window > New reader window to open an empty reader, or Story > Open in new reader window to open another reader for the current article. Docked readers keep their articles when you select another story from the sidebar. Minimized readers can be restored and reused for their site. A window's Next story link reads on down the list in the same window.

    A link in a comment or an article to another Hacker News or Lobsters thread opens that thread in a window too. A link to a single HN comment opens the comment above its replies.

    While a story's window is in front, the Story and Discussion menus in the menu bar hold its commands: pinning, Next story, sorting, collapsing every thread, jumping to the first new comment. Each window also remembers where you were in its article and discussion, so a reload, or Back to a story that Next took you past, finds your place again. Closing a window forgets it.

    The whole arrangement lives in the address, so a bookmark or a shared link brings it back, and Back undoes the last change. Moving between Hacker News, Lobsters, their lists, Pinned and Find changes only the list, and leaves the windows open.

    The list

    The pin at the start of a row keeps the story in Pinned, and the cross at its end hides it. Pinned can be narrowed by words in the title, site or author, by source, and to the stories you haven't opened yet, and ordered by when you pinned them, by points or by comments; the filters are part of the address, so a filtered view can be bookmarked. Scroll past the end of the list to load more. Domain filters, in the View menu, hide every story from a site.

    Collections are named lists of stories. Story > Add to collection files the story in front into one or more of them, and the Collections feed shows them all or one at a time; the menu that chooses collections also creates, renames, and deletes them. A note is your own text on a story. Choose Add note in a story's toolbar to write one; it saves as you type. Rows with a note carry the note mark, and the Notes feed lists every noted story and searches the text of your notes.

    Browsing view

    View > Windowed and View > Classic select the browsing view and save your default in this browser. Window view reuses a reader for each feed. Classic view opens stories and applets as pages. Open as a page is a one-off action that does not change your saved default. Use the Windowed selector to return an article to a window. Direct page links always open as pages.

    Applets

    The Applets menu in the menu bar holds three tools, each a window of its own. Replies to me takes your Hacker News user name and lists the replies to your last thirty comments and stories, checking again every three minutes while it is open, and marking what is new since you last marked them read. Look up a user opens a profile on Hacker News or Lobsters, with their submissions and recent comments, as a commenter's name in any discussion does; the bar at the top of a profile looks up someone else in the same window, and Back returns to the one before. Who is hiring? filters the posts of HN's monthly hiring threads by the words you type.

    They read only what the sites publish to everyone, so none of them asks for a login, and your user name stays in this browser anonymously and is included in retained applet state when signed in.

    Find

    Find takes any link and lists every time it was submitted to Hacker News and Lobsters, so you can read each discussion of it.

    About

    YAVCHN never sees your Hacker News or Lobsters login. The discussion is fetched from each site's public API; to vote or reply, follow the link above the discussion, or the arrow beside a comment, to the source's own site. Without a YAVCHN account, your data stays in this browser. When signed in, pins, collections, notes, blocked domains, and retained reading state are stored with your account and synchronized across devices. Hidden stories and layout stay in this browser.

    Open source: github.com/paulmooreparks/yavchn. Built with PUDL.