Representative Line: Changing With the Times
by Remy Porter from The Daily WTF on (#30G32)
Melody got tapped to do a code review on a pull-request from a veteran team-member. It was" an interesting PR, in that very, very little changed. The code was terrible before anyone touched it- for example, the C-file started with 355 lines of variable declarations inside of the main method.
It was, in fact, down around line 354 where Melody noticed the change.
-int right_now_month = 6; // TODO: look up based on current state+int right_now_month = 7; // TODO: look up based on current state
Out of morbid curiosity, Melody went back through the history, and sure enough, once a month, there was a PR that involved incrementing right_now_month.
[Advertisement] Incrementally adopt DevOps best practices with BuildMaster, ProGet and Otter, creating a robust, secure, scalable, and reliable DevOps toolchain.