# code style Coding conventions intended to improve code readability and consistency. This is one page of public article previews, not the complete archive. Follow Next page to continue. Summaries are not the original full articles. ## Kotlin Code Review: Avoiding Unproductive Nitpicking Over Concise and Idiomatic Code DevFeed: [Kotlin Code Review: Avoiding Unproductive Nitpicking Over Concise and Idiomatic Code]() Original publisher: [Read original article]() Published: 2020-10-14T00:00:00Z Content type: tutorial Language: en Sources: [Vasya Drobushkov]() Topics: [Code review](), [Kotlin](), [code style](), [Development]() Tags: [code-review](), [code-style](), [development-process](), [kotlin](), [quality]() ### AI overview This article examines how code review can become unproductive when reviewers focus excessively on style, concision, or idiomatic Kotlin patterns. It discusses the trade-offs behind some Kotlin constructs and argues that shorter code is not automatically better. ### Source excerpt Introduction Code review is an essential tool for a development team, which helps ensure high-quality standards of code. While reviewing others' code one might find bugs, design issues, and so on. One another part of reviewing is getting used to the codebase and following team's code style for better maintenance. Though it is pretty cool in theory, in practice team might face few issues, one of which is nitpicking. When review is bloated with dozens of similar comments related to e.g. how beautifully code looks like. Common code style is important, but having a lot of similar comments in each review doesn't help. Instead of trying to understand what code is doing, nitpicker writes a lot of similar comments on the style. Back and forth discussions or fixes of such slow down development process and overall make team's morale worse. ## Empire Node 2016 DevFeed: [Empire Node 2016]() Original publisher: [Read original article]() Author: Jack Tarantino Published: 2016-11-08T22:57:34Z Content type: article Language: en Sources: [Jacopo Tarantino]() Topics: [code style](), [ESLint](), [React](), [Redux](), [MongoDB](), [Promise](), [CommonJS](), [middleware]() Tags: [callback](), [code](), [code-style](), [commonjs](), [empire-node](), [eslint](), [javascript](), [modules](), [mongodb](), [new-york-city](), [node-js](), [react](), [redux]() ### AI overview Notes from Empire Node 2016 cover talks about HyperTerm and its React and Redux plugin architecture, ESLint rules and coding practices, and MongoDB, along with other programming topics. ### Source excerpt Yesterday I had the good fortune to attend Empire Node at The National Museum of the American Indian, courtesy of InRhythm. A good time was had by all and we got to see some great talks! There was programmable music, smart ways to get into code style linting and, of ## How Code Reviews Support Chromium's Code Quality and Team Resilience DevFeed: [How Code Reviews Support Chromium's Code Quality and Team Resilience]() Original publisher: [Read original article]() Author: Monica Dinculescu Published: 2014-03-31T00:00:00Z Content type: opinion Language: en Sources: [Monica Dinculescu]() Topics: [Code](), [Chromium](), [code style](), [bus factor]() Tags: [bus-factor](), [chromium](), [code](), [code-reviews](), [code-style]() ### AI overview The article explains how Chromium uses code reviews and style guides to manage a large, actively changing codebase. It argues that review helps catch broken or poor-quality code, maintain consistency, and reduce reliance on a small number of contributors. ### Source excerpt Stats: a preamble I've been reading too much about March Madness brackets, so I thought I had to run some numbers around here like the cool kids do. Get your umbrella out, it's about to rain cold facts. In the history of time, Chromium has had 205,095 commits made by 1,943 contributors representing 7,431,088 lines of code. In the last 30 days, there have been 5021 commits, by 637 contributors, including 53 new hoomans. I did some advanced Nate Silver analysis here for you, and that's at least 167 commits and 1+ new committers a day. On average, that's at least 7 commits an hour. Every hour. All of the hours. That's an imperial ton of new code being added, by what it seems like new people. Imagine if everyone could commit code willy-nilly. Are you imagining a minefield? You should. Code reviews ftw Good news for our browser using audience! Chromium isn't a minefield, and on top of it, has pretty awesome looking code. This comes from the fact that any code changes need to be reviewed and blessed before they can land on the master branch. More eyes means less bugs means you're less likely to commit broken code and break the internet. And you really don't want to break the internet. Even if you have tests, and everything is going your way, you can write correct, but genuinely shitty code. 7 million lines of kinda-shitty code is not something anyone wants to work with, and are worth investing a little time in fixing. Code reviews also bring up the bus factor, which is my favourite sinister nerd metaphor. You know, the buuuuuus factor. The number of people that can get run over by a bus on a team before that team is royally and epically screwed. If all the code that you write has been closely read by a different person, then you're probably ok getting run over by a bus every once in a while. But still, you probably shouldn't. Who would feed your cat? Consistent code is the best code Code style guides are sooper neat, and are a huge part of code reviews, because ain't nobo ## C++ Practices from Chromium Code Style: Avoiding Copies and Using Debug Checks DevFeed: [C++ Practices from Chromium Code Style: Avoiding Copies and Using Debug Checks]() Original publisher: [Read original article]() Author: Monica Dinculescu Published: 2014-01-20T00:00:00Z Content type: tutorial Language: en Sources: [Monica Dinculescu]() Topics: [C++](), [Chromium](), [code style](), [Compiler](), [debug](), [test-coverage]() Tags: [c-plus-plus](), [chromium](), [clang](), [code-style](), [compiler](), [debug](), [test-coverage]() ### AI overview A practical C++ article based on Chromium code style. It discusses preventing unwanted copy construction and assignment with a macro, and using debug-only checks to catch bad scenarios during development and testing. ### Source excerpt Disclaimer: these aren't new protipz. I didn't make them up. They're actually straight out of the Chromium code style, they're pretty trivial, and you might already use them. But just in case you're not a Chromium committer (the outrage), or are fairly new at C++ and want to make your code less suck, here they are. I think they're neat. Copy constructors and their brethren You know that scene from The Fly when Jeff Goldblum, having not screwed up teleporting a small baboon, decides he should totally teleport himself? But then he screws that up (because software), manages to turn himself into a giant terrifying fly (because David Cronenberg), and continues to give me nightmares as an adult. That's exactly how I feel about copy constructors. You can absolutely get them right, but they're a pain, and among other crimes they're committing, they're sometimes deceivingly slow. The point is, most of the time you don't even need them. I mean, Jeff Goldblum teleported himself like three meters away. Couldn't he have just walked? What we tend to do instead is convince the compiler to get annoyed with us if we try to use a copy constructor. This is easy because the compiler <3s being annoyed with us. So we can define a nice macro (stay with me) that adds a private declaration, but doesn't implement it: #define DISALLOW_COPY_AND_ASSIGN(TypeName) \ TypeName(const TypeName&); \ void operator=(const TypeName&) Which you would then use in your private section of your class, like so: class Hooman { public: Hooman(); ~Hooman(); private: DISALLOW_COPY_AND_ASSIGN(Hooman); }; Now, when you try to be ambitious and clone Jeff Goldblum, Hooman jeffGoldblum; Hooman teleportedJeffGoldblum(jeffGoldblum); Clang will tell you something like "error: calling a private constructor of class 'Hooman'". Other compilers might tell you other things, but they'll generally have the same annoyed tone. Now would be a good time to apologize to your compiler for all the silly things you've done in the past. ## October 2013 Meeting Minutes DevFeed: [October 2013 Meeting Minutes]() Original publisher: [Read original article]() Published: 2013-11-14T00:00:00Z Content type: news Language: en Sources: [Front Page on ReactOS Website]() Topics: [ReactOS](), [Development](), [code style]() Tags: [code-style](), [development](), [free](), [open-source](), [os](), [react](), [reactos](), [win32](), [winapi]() ### AI overview Minutes from the ReactOS project's October 31, 2013 meeting cover team coordination, adoption of a coding style guideline, and the availability of foundation funds for development hardware. ### Source excerpt 2013-10-31 19:00 UTC dev.reactos.org, #meeting Proceedings Meeting started at 19:08 by Aleksey Bragin. Point 1: CoordinationPoint 2: Code Style GuidelinesPoint 3: Donations and HardwarePoint 1Aleksey Bragin opened up the meeting with a statement that the project's members need to coordinate and work better together to avoid internal confusion as to what work is being done and what work requires some help due to other commitments by team members. Point 2Colin Finck presented for approval a finalized draft for the project's coding style guideline. ## Tip for 1-click indentation settings changes in Visual Studio DevFeed: [Tip for 1-click indentation settings changes in Visual Studio]() Original publisher: [Read original article]() Author: Andrew Khmylov Published: 2011-02-27T00:00:00Z Content type: tutorial Language: en Sources: [Despite the odds]() Topics: [Visual Studio](), [code style](), [Automation](), [ide](), [C#]() Tags: [automation](), [change](), [code](), [code-style](), [csharp](), [ide](), [macros](), [spaces](), [style](), [visual-studio]() ### AI overview A tutorial explains how to create a Visual Studio macro that toggles the C# indentation setting between tabs and spaces. It uses the DTE TextEditor CSharp InsertTabs property and provides keyboard-accessible macro commands. ### Source excerpt I work at several projects at a time, and I have different settings/requirements concerning the code style for each of them. The main issue is 'Tabs vs. Spaces' option - the first project requires using tabs, the second - spaces. It's obvious that going Tools -> Settings -> ... is not an option for everyday use. Sounds like a good task for the automation macros, right? Navigate to Tools -> Macros -> Macros IDE (or use Alt+F11) Create new macros in Project Explorer (Project - Add Module) Write a method that changes the indentation settings: Public Module TabsModule Public Sub ToggleTabs() Dim currentSetting As Boolean = DTE.Properties("TextEditor", "CSharp").Item("InsertTabs").Value DTE.Properties("TextEditor", "CSharp").Item("InsertTabs").Value = Not currentSetting End Sub End Module Now you can use Macros Explorer (Alt+F8, also you can dock it as one of your panels) to quickly change settings (by double-clicking on the corresponding item name in the macros tree)