I ended up firing a guy because he would not stop littering the code with crap like:
// 2 lines added by I. Dee Ott Feb 12, 2005 // increment i i++; // 1 line removed by I. Dee Ott, Feb 12, 2005 // i = i + 1;
// this function can only be run by admin
Which would make sense, except that it's the code itself said to indicate this. Or, does he mean for that to be documented in the RCS?