Why didn't he replace each one of: "// 10 repetitive lines of math" with: "doRepetitiveMath()" ?

Sounds like he did a lot more than eliminate duplication, and perhaps that's what his boss was unhappy about.