Remove melee_train() duplicate

[CodeBandit]

Summary

SUMMARY: bugfixes "Remove duplicate melee_train() call"

Purpose of change

Remove a duplicate melee_train() call

Describe the solution

This is being called when just a few lines down, the picture after it is being called as well.

Describe alternatives you've considered

not removing it

Testing

no compilation errors

Additional context

Additional things it affects:
Focus drain will not drain as quickly for melee combat (actually a buff?) so there could be more exp for melee for levels 1 to 5.

Focus drain becomes consistent. Previously, it was faster from levels 1 to 5 than it was 6 to 10. This was inconsistent with how every other skill trained.

[anothersimulacrum]

This is not a duplicate. The one removed is only called if hit_spread < 0 (a miss), the other one is only called if hit_spread >= 0 (a hit).

[CodeBandit]

This is not a duplicate. The one removed is only called if hit_spread < 0 (a miss), the other one is only called if hit_spread >= 0 (a hit).

wow yikes, i can't believe I missed that. Closing the PR