Code diving, documentation correction for criticals


#1

Doesn’t affect gameplay at all, but the example cases for calculating critical hits in the code documentation has an error. The dexterity / perception “roll” (one of three rolls that work together) for a character with stats 20/20/20/20 would be at 85% not 100%. It starts at 25% and you would get an extra 20 from the Dex and an extra 40 from the Per.

Would be nice if someone could correct it just for better clarity.


#2

If you have enough knowledge to dive through the code to figure that you, you can open a PR with the fix yourself. This is an open source project and everyone is welcome to contribute.


#3

Ha, I’m not there yet. If I was already able to do that stuff it would be done, I’d have skipped the forum post. :slightly_smiling_face:


#4

How hard is it to edit a comment string and open a PR? You can do it entirely through github’s web interface if you don’t feel up to dealing with git on your personal machine.


#5

Glad to hear it’s easy once you know how. Don’t know what a PR thingy is yet, but I’m sure I can figure it out at some point.

Happy Thanksgiving good sir >salute<


#6

Check out https://github.com/help and https://github.com/CleverRaven/Cataclysm-DDA/blob/master/.github/CONTRIBUTING.md


#7

Thanks Kevin, much appreciated