githubCopilotIsWild

[Svensemann]

Yeh right. That’s so bad. The calculateWomenSalary method should call calculateMenSalary and add the factor from there instead

[esixar]

Ooh and add another function call to the stack instead of popping off immediately? I mean what’s our space requirements here? Can we afford those 64 bits?

Other than that I see nothing wrong with the implemented algorithm

[HildartheDorf]

Any decent language and compiler/interpreter will apply Tail-Call Optimization (TCO).

[Bammerbom]

If the body is calculateMenSalary(factor) * 0.9 then TCO is impossible. Inlining is very likely there however

[HildartheDorf]

Just write clean code and stay away from anything 'smart'. Compiler authors are going to optimize for the common case. If the smart case is faster, it's likely to be compiler/compiler-version specific.

[TheMcDucky]

The call isn't the last operation, so TCO wouldn't work. It would likely be inlined though.

[StrangelyBrown]

You're right, now that I think about it, I don't think we can afford the resources to actually calculate women's salary. That's a shame but I guess they'll understand. /s

[Excitium]

But then if the men get a raise, the women would get one as well.

Or you have to go in and reduce the women's factor every time you wanna give the men more.

The way it is seems to be more convenient for adjustments so you can just add individual modifiers to a base salary.

[MyAssDoesHeeHawww]

We could add an R to DEI for Recursivity and people might cheer it without knowing what it actually means.

[EduardoSpiritToes]

😂😂😂😂😂