Tuesday, December 20th, 2011
Recently I’ve been noticing a surprising pattern in code I’m reviewing for the kernel. A lot of people seem to have taken to writing code that I’d expect to look like this: switch (thing) { case VALUE: /* Stuff */ break; case BAR: /* Nonsense */ break; default: /* Whatever */ break; } with if [...]
Friday, September 9th, 2011
One of the big things that seems to cause a learning curve for many new contributors for Linux and other projects that make a big effort with code review is the process of putting patches together in a way that makes the code review process more smoothly. This is a fairly straightforward thing but it [...]
Filed in ASoC, Linux, Planet Debian, regmap, regulator, tech
|
Also tagged code review, development, gerrit, git, Linux, process
|
Permalink
|