Banned C++ features in Chromium

(chromium.googlesource.com)

107 points | by szmarczak 5 hours ago ago

89 comments

  • Night_Thastus 4 hours ago

    Nothing particularly notable here. A lot of it seems to be 'We have something in-house designed for our use cases, use that instead of the standard lib equivalent'.

    The rest looks very reasonable, like avoiding locale-hell.

    Some of it is likely options that sand rough edges off of the standard lib, which is reasonable.

    • ryandrake 2 hours ago

      > We have something in-house designed for our use cases, use that instead of the standard lib equivalent

      Yea, you encounter this a lot at companies with very old codebases. Don't use "chrono" because we have our own date/time types that were made before chrono even existed. Don't use standard library containers because we have our own containers that date back to before the STL was even stable.

      I wonder how many of these (or the Google style guide rules) would make sense for a project starting today from a blank .cpp file. Probably not many of them.

      • LexiMax 6 minutes ago

        > I wonder how many of these (or the Google style guide rules) would make sense for a project starting today from a blank .cpp file. Probably not many of them.

        The STL makes you pay for ABI stability no matter if you want it or not. For some use cases this doesn't matter, and there are some "proven" parts of the STL that need a lot of justification for substitution, yada yada std::vector and std::string.

        But it's not uncommon to see unordered_map substituted with, say, sparsehash or robin_map, and in C++ libraries creating interfaces that allow for API-compatible alternatives to use of the STL is considered polite, if not necessarily ubiquitous.

      • mtklein 2 hours ago

        ... and for many of those blank .cpp file projects, the style guide should probably just read "start a blank .rs file instead."

        • acdha 36 minutes ago

          Look, I even share your language preference but this is still unnecessary.

          • galangalalgol 3 minutes ago

            Are there really any good reasons to start a brand new project in c++ though? No one who can write modern c++ has any trouble with rust in my experience, and all the other common options are even quicker to pick up. Creating bindings isn't hard anymore if your niche library doesn't have any yet. Syntactic preference I guess, but neither c++ or rust are generally considered elegant or aesthetic choices.

    • vitaut 2 hours ago

      Somewhat notable is that `char8_t` is banned with very reasonable motivation that applies to most codebases:

      > Use char and unprefixed character literals. Non-UTF-8 encodings are rare enough in Chromium that the value of distinguishing them at the type level is low, and char8_t* is not interconvertible with char* (what ~all Chromium, STL, and platform-specific APIs use), so using u8 prefixes would obligate us to insert casts everywhere. If you want to declare at a type level that a block of data is string-like and not an arbitrary binary blob, prefer std::string[_view] over char*.

      • ChrisSD 2 hours ago

        `char8_t` is probably one of the more baffling blunders of the standards committee.

        • jjmarr an hour ago

          there is no guarantee `char` is 8 bits, nor that it represents text, or even a particular encoding.

          If your codebase has those guarantees, go ahead and use it.

          • 20k an hour ago

            char8_t also isn't guaranteed to be 8-bits, because sizeof(char) == 1 and sizeof(char8_t) >= 1. On a platform where char is 16 bits, char8_t will be 16 bits as well

            The cpp standard explicitly says that it has the same size, typed, signedness and alignment as unsigned char, but its a distinct type. So its pretty useless, and badly named

          • dataflow an hour ago

            How many non-8-bit-char platforms are there with char8_t support, and how many do we expect in the future?

  • bengoodger an hour ago

    Seeing the comments here talking about ancient codebases got me wistful for when Chromium was new and everything seemed possible.

    That triggered a flash of feeling extremely old realizing we broke ground on this codebase 20 years ago this year!

  • dnmc 3 hours ago

    There are yet more interesting docs in the parent directory :)

    https://chromium.googlesource.com/chromium/src/+/main/styleg...

  • lateforwork 2 hours ago

    You almost never see a list of banned Java features (or even banned C# features). On the other hand any serious C++ development team is going to have a list of banned features. Java eliminated the features that you would want to ban.

    • oldmanhorton 24 minutes ago

      In C#, you would normally implement rules like this with a custom Roslyn Analyzer or with https://github.com/dotnet/roslyn/blob/main/src/RoslynAnalyze.... It’s fair to say C# projects tend to have smaller denylists than mature C++ projects, but banned APIs definitely exist in mature C# projects.

    • nilamo 2 hours ago

      This seems factually incorrect and ignorant of history. Java has tons of things which shouldn't be used. Serialization (use Jackson now, not the built-in stuff), date/time (there's an entirely different namespace so you don't accidentally use garbage classes), etc.

      C# similarly has old warts that are discouraged now. .NET Framework is a great example (completely different from modern c#, which used to be called "dotnet core"). WPF and MAUI are also examples. Or when "dynamic" was used as a type escape hatch before the type system advanced to not need it. ASP being incompatible with ASP.NET, the list goes on.

      They're just languages, there's no reason to pretend they're perfect.

      • twisteriffic an hour ago

        > C# similarly has old warts that are discouraged now. .NET Framework is a great example (completely different from modern c#, which used to be called "dotnet core"). WPF and MAUI are also examples. Or when "dynamic" was used as a type escape hatch before the type system advanced to not need it. ASP being incompatible with ASP.NET, the list goes on.

        Almost all of this is incorrect or comparing apples to oranges.

        .net framework and .net core are runtime and standard library impl, not languages. C# is a language that can target either runtime or both. Framework is still supported today, and you can still use most modern C# language features in a project targeting it. WPF and Maui are both still supported and widely used. ASP predates .net - c# was never a supported language in it. ASP.net core has largely replaced ASP.net, but it's again a library and framework, not a language feature.

        Dynamic in c# and the dlr are definitely not widely used because it's both difficult to use safely and doesn't fit well with the dominant paradigm of the language. If you're looking for STD lib warts binaryserializer would have been an excellent example.

      • lateforwork an hour ago

        Those are libraries not language features.

        • plorkyeran an hour ago

          So is nearly all of this list.

          • lateforwork an hour ago

            There are clearly demarcated language features sections and library sections. That definitely makes sense for C++, it is a poorly designed language and you definitely have to know what features to avoid.

        • bathtub365 an hour ago

          dynamic is a language feature

    • jameslars 41 minutes ago

      I think Java has plenty of features that end up implicitly banned at least. e.g. you will really never see a `java.util.Vector` in any modern code base. No one `implements Serializable` anymore in practice. `Objects.equals()` and `Objects.hashCode()` get you through 99% of equals/hash code implementations. The list goes on.

      I guess the difference is it's rarely "dangerous" or "hard to reason about" using the old features unlike what I see in the C++ list. Java replaces things with better things and momentum shifts behind them kind of naturally because the better things are objectively better.

    • AdieuToLogic an hour ago

      > You almost never see a list of banned Java features ...

      The instanceof[0] operator is typically banned from use in application code and often frowned upon in library implementations.

      0 - https://docs.oracle.com/javase/specs/jls/se7/html/jls-15.htm...

      • twic 41 minutes ago

        I've never heard of that being banned. It hasn't been banned anywhere i've written Java.

    • madeofpalk 38 minutes ago

      I would imagine most codebases, even in modern languages, tend to have a list of banned features. Typically you use a linter to catch these.

    • MBCook an hour ago

      As a Java programmer I can only think of one thing:

      Reflection - unless you really need to do something fancy almost certainly a very bad idea in normal application code.

      Other than that it’s either just limiting yourself to a specific JVM version or telling people not to use old syntax/patterns that were replaced with better options long ago.

  • bfrog 27 minutes ago

    This list is longer than the features in all of C I feel like at first glance. Wow that is overwhelming.

  • Tempest1981 3 hours ago

    Where does it list the preferred alternatives to banned features?

    For example:

    > The <filesystem> header, which does not have sufficient support for testing, and suffers from inherent security vulnerabilities.

    • comex 3 hours ago

      For most of the banned library features, the preferred alternative is listed right there in the notes. <filesystem> is one of the exceptions.

    • jeffbee 3 hours ago

      base/files

    • TheRealPomax 3 hours ago

      Gonna venture a guess and say probably https://www.chromium.org/developers, as that's where all the information for folks who actually need to know that kind of thing lives.

  • ddtaylor 4 hours ago

    Exceptions are banned, but an exception is made for Windows.

  • weinzierl an hour ago

    I'd curious about the banned Rust features. Surely, Rust has at lot fewer foot guns, but it isn't that there aren't any.

    • jtrueb an hour ago

      Heard of `#![forbid(unsafe_code)]` ?

  • nixosbestos 12 minutes ago

    Rust is too complicated!!11 Ooops wrong thread.

  • amelius 3 hours ago

    Is there a way to make this formal, like in the code, making the compiler complain when you try to use these features?

    • 6r17 2 hours ago

      I'm not a c++ user but i'm pretty sure you should be able to pull-off a macro to do that ; in c you could alias the lib for something that breaks + alert ; I don't know how I would integrate such additional compiler checks in rust for other kinds of rules however - it's interesting to think about

    • TingPing 2 hours ago

      It’s a bit different, but WebKit uses Clangs static analysis.

    • loeg 2 hours ago

      It is relatively easy to check these things with static analyzers, if nothing else.

    • mkoubaa an hour ago

      Kythe can be used for that

  • dfajgljsldkjag 4 hours ago

    The banned list proves that context matters more than having the newest tools. These features work well for small apps but they cause problems in a project this size.

    • trinix912 2 hours ago

      IIRC a big part of Google’s coding guidelines is also about making it easy for people not heavily invested in a specific language to contribute safely. So not necessarily a project size but rather an organizational concern.

      They’d rather see it done the same way it would’ve been in any other similar language than with a language specific feature.

      There are also portability concerns in mind given that projects like Chromium have to be easily portable across a vast amount of platforms (this shows with things like long long which is also on the list).

      • jonstewart an hour ago

        Go is an extremely cynical language in this regard.

    • loeg 2 hours ago

      Some of it is historical reasons or portability more than anything else. Chrome is an old C++ project and evolved many of its own versions of functionality before standardization; and there's benefit to staying on its own implementations rather than switching.

  • kazinator an hour ago

    Good call on those u8"..." literals.

    Source code should all be UTF-8 natively, letting you directly write UTF-8 text between quotes.

    Exactly their rationale.

    These literals are a solution in search of a problem ... which is real but has a much better solution.

  • jesse__ 4 hours ago

    It's remarkable to me how many codebases ban exceptions and yet, somehow, people still insist they're good.

    • BeetleB 4 hours ago

      > Our advice against using exceptions is not predicated on philosophical or moral grounds, but practical ones. ... Things would probably be different if we had to do it all over again from scratch.

      They are clearly not against them per se. It simply wasn't practical for them to include it into their codebase.

      And I think a lot of the cons of exceptions are handled in languages like F#, etc. If f calls g which calls h, and h throws an exception, the compiler will require you to deal with it somehow in g (either handle or explicitly propagate).

      • jesse__ 4 hours ago

        My issue with exceptions is also practical. If they didn't introduce significant stability issues, I'd have no problem. As it stands, it's impossible to write robust software that makes use of C++ exceptions.

        > the compiler will require you to deal with it somehow in g

        I agree, this is the sensible solution.

        • yunnpp 2 hours ago

          What stability issues?

      • jandrewrogers 3 hours ago

        In low-level systems software, which is a primary use case for C++, exceptions can introduce nasty edge cases that are difficult to detect and reason about. The benefits are too small to justify the costs to reliability, robustness, and maintainability.

        Exceptions in high-level languages avoid many of these issues by virtue of being much further away from the metal. It is a mis-feature for a systems language. C++ was originally used for a lot of high-level application code where exceptions might make sense that you would never use C++ for today.

        • BeetleB 3 hours ago

          > In low-level systems software, which is a primary use case for C++

          I can assure you: Most C++ SW is not written for low-level.

          > exceptions can introduce nasty edge cases that are difficult to detect and reason about.

          That's true, except for languages that ensure you can't simply forget that something deep down the stack can throw an exception.

          BTW, I'm not saying C++'s exceptions are in any way good. My point is that exceptions are bad in C++, and not necessarily bad in general.

          • beached_whale 3 hours ago

            The model of communicating errors with exceptions is really nice. The implementation in C++ ABI's is not done as well as it could be and that results in large sad path perf loss.

          • jandrewrogers 2 hours ago

            > That's true, except for languages that ensure you can't simply forget that something deep down the stack can throw an exception.

            Sometimes it is not safe to unwind the stack. The language is not relevant. Not everything that touches your address space is your code or your process.

            Exception handlers must have logic and infrastructure to detect these unsafe conditions and then rewrite the control flow to avoid the unsafety. This both adds overhead to the non-exceptional happy path and makes the code flow significantly uglier.

            The underlying cause still exists when you don't use exceptions but the code for reasoning about it is highly localized and usually has no overhead because you already have the necessary context to deal with it cleanly.

        • kllrnohj 3 hours ago

          If you forget to handle a C++ exception you get a clean crash. If you forget to handle a C error return you get undefined behavior and probably an exploit.

          Exceptions are more robust, not less.

          • nomel 3 hours ago

            Yeap. forgetting to propagate or handle an error provided in a return value is very very easy. If you fail to handle an exception, you halt.

        • drnick1 2 hours ago

          > In low-level systems software, which is a primary use case for C++

          I don't this this is true. There is A LOT of C++ for GUI applications, video games, all kind of utilities, scientific computing and others. In fact, I find that the transition to "modern" alternatives from native GUI toolkits in C/C++ has led to a regression in UI performance in general. Desktop programs performed better 20 years ago when everything was written in Win32, Qt, GTK and others and people did not rely on bloated Web toolkits for desktop development. Even today you can really feel how much more snappy and robust "old school" programs are relative to Electron and whatnot.

          • saghm 2 hours ago

            To clarify, you think that low-level systems software is only a secondary use case for C++? The part you quoted does not make claims about whether there are other primary use cases, just that low-level systems software is one of them, so it's not clear why it being useful elsewhere is a rebuttal of that.

        • beached_whale 3 hours ago

          C++ exceptions are fast for happy path and ABI locked for sad path. They could be much faster than they are currently. Khalil Estell did a few talks/bunch of work on the topic and saw great improvements. https://youtu.be/LorcxyJ9zr4

        • senfiaj 3 hours ago

          > "In low-level systems software, which is a primary use case for C++, exceptions can introduce nasty edge cases that are difficult to detect and reason about. The benefits are too small to justify the costs to reliability, robustness, and maintainability."

          Interestingly, Microsoft C / C++ compiler does support structured exception handling (SEH). It's used even in NT kernel and drivers. I'm not saying it's the same thing as C++ exceptions, since it's designed primarily for handling hardware faults and is simplified, but still shares some core principles (guarded region, stack unwinding, etc). So a limited version of exception handling can work fine even in a thing like an OS kernel.

          • jandrewrogers 2 hours ago

            FWIW, I think it is possible to make exception-like error handling work. A lot of systems code has infrastructure that looks like an exception handling framework if you squint.

            There are two main limitations. Currently, the compiler has no idea what can be safely unwound. You could likely annotate objects to provide this information. Second, there is currently no way to tell the compiler what to do with an object in the call stack may not be unwound safely.

            A lot of error handling code in C++ systems code essentially provides this but C++ exceptions can't use any of this information so it is applied manually.

        • matheusmoreira 3 hours ago

          Exceptions are actually a form of code compression. Past some break even point they are a net benefit, even in embedded codebases. They're "bad" because the C++ implementation is garbage but it turns out it's possible to hack it into a much better shape:

          https://youtu.be/LorcxyJ9zr4

          • secondcoming 3 hours ago

            There is no such thing as the 'C++ implementation' of exceptions. Each vendor can do it differently.

      • jayd16 3 hours ago

        Is this correct? I don't know F# but I thought it had unchecked exceptions. How does it handle using C# libs that throw unchecked exceptions?

      • heyitsdaad 4 hours ago

        The “pros” list is exceptionally weak. This was clearly written by someone who doesn’t like exceptions. Can’t blame them.

    • azov 3 hours ago

      Most codebases that ban exceptions do it because they parrot Google.

      Google’s reasons for banning exceptions are historical, not technical. Sadly, this decision got enshrined in Google C++ Style Guide. The guide is otherwise pretty decent and is used by a lot of projects, but this particular part is IMO a disservice to the larger C++ ecosystem.

      • alextingle 3 hours ago

        I agree. I've worked on large C++ code bases that use exceptions, and they've never caused us any real problems.

    • ryandrake 3 hours ago

      I think reasonable people can disagree about whether C++ exceptions are "good" or not.

      There are things you can't do easily in C++ without using exceptions, like handling errors that happen in a constructor and handling when `new` cannot alloc memory. Plus, a lot of the standard library relies on exceptions. And of course there's the stylistic argument of clearly separating error-handling from the happy-path logic.

      I won't argue that it's popular to ban them, though. And often for good reasons.

      • canucker2016 36 minutes ago

        For exception-less C++, you'd declare an operator new() that doesn't throw exceptions and just returns NULL on allocation failure along with a simple constructor and a followup explicitly-called init() method that does the real work which might fail and returns an error value on failure.

    • tester756 4 hours ago

      They're good for exceptional situations where foundamental, core assumptions are broken for some reason.

      In such scenario there's no error recovery, software is expected to shutdown and raise loud error.

      • jesse__ 4 hours ago

        If you're planning on shutting down, what's the fundamental difference between throwing an exception, vs simply complaining loudly and calling exit() ..?

        • trinix912 4 hours ago

          Sometimes it’s useful to handle the exception somewhere near its origin so you can close related resources, lockfiles, etc. without needing a VB6 style “On Error GoTo X” global error handler that has to account for all different contexts under which the exceptional situation might have occurred.

          • PhilipRoman 3 hours ago

            Your process can crash or be killed at any moment anyway. Depending on in-band cleanup is not reliable.

            • nomel 3 hours ago

              Sure, but there are many cases where you don't have to halt because you can cleanup and carry on.

          • matheusmoreira 3 hours ago

            > a VB6 style “On Error GoTo X” global error handler that has to account for all different contexts under which the exceptional situation might have occurred

            ... That seems like a pretty accurate description of how exception handling mechanisms are implemented under the hood. :)

        • einpoklum 2 hours ago

          The code that's throwing an exception typically does not know that the exception catcher will shut anything down.

          And - very often, you would _not_ shut down. Examples:

          * Failure/error in an individual operation or action does not invalidate all others in the set of stuff to be done.

          * Failure/error regarding the interaction with one user does not mean the interaction with other users also has to fail.

          * Some things can be retried after failing, and may succeed later: I/O; things involving resource use, etc.

          * Some actions have more than one way to perform them, with the calling code not being able to know apriori whether all of them are appropriate. So, it tries one of them, if it fails tries another etc.

      • spacechild1 2 hours ago

        > They're good for exceptional situations where foundamental, core assumptions are broken for some reason.

        No, that's what assertions or contracts are for.

        Most exceptions are supposed to be handled. The alternative to exceptions in C++ are error codes and `std::expected::. They are used for errors that are expected to happen (even if they may be exceptional). You just shouldn't use exceptions for control flow. (I'm looking at you, Python :)

      • ljm 3 hours ago

        Yet, if you can only explain an exception using the word ‘exception’ you’re not making any head way.

        I like the idea of an exception as a way to blow out of the current context in order for something else to catch it and handle in a generic manner. I don’t like the idea of an exception to hide errors or for conditional logic because you have to know what is handling it all. Much easier to handle it there and then, or use a type safe equivalent (like a maybe or either monad) or just blow that shit up as soon as you can’t recover from the unexpected.

      • dijit 4 hours ago

        I use asserts for this purpose.

    • wvenable 4 hours ago

      Looking at this ban list, they've removed everything from C++ that makes it fun. Come on people, who doesn't love a little std::function?!?

      On banning exceptions:"Things would probably be different if we had to do it all over again from scratch."

      https://google.github.io/styleguide/cppguide.html#Exceptions

  • einpoklum 2 hours ago

    Since Chromium stopped allowing manifest-v2 extensions, i.e. significantly crippled what extensions can do and made it impossible to use some key extensions like uBlock Origin, I've decided to avoid it.

    Anyway, about these C++ conventions - to each software house its own I guess. I don't think banning exceptions altogether is appropriate; and I don't see the great benefit of using abseil (but feel free to convince me it's really that good.)

    • jsheard an hour ago

      If nothing else Abseil gives you state-of-the-art hashmaps that run circles around the STL ones, which are slow pretty much by definition. The spec precludes the modern ways of implementing them.

  • WalterBright 3 hours ago

    Modules are banned - they should have just copied D modules.

  • grougnax 2 hours ago

    C++ itself should be forever banned