diasurgical / devilutionX

Diablo build for modern operating systems
Other
8.04k stars 790 forks source link

Refactor devilution.DefaultClass::CheckInvPaste(int, int, int) #1996

Open AJenbo opened 3 years ago

AJenbo commented 3 years ago

I've selected devilution.DefaultClass::CheckInvPaste(int, int, int) for refactoring, which is a unit of 362 lines of code and 135 branch points. Addressing this will make our codebase more maintainable and improve Better Code Hub's Write Simple Units of Code guideline rating! 👍

Here's the gist of this guideline:

You can find more info about this guideline in Building Maintainable Software. 📖


Good luck and happy coding! :shipit: :sparkles: :100:

matheusgomes28 commented 7 months ago

Created #6984, need some initial feedback + figure out how to run the failing test locally so I can debug it 😄