widelands-dev team mailing list archive
Mailing list archive
Re: [Merge] lp:~widelands-dev/widelands/handling-various-fs-errors into lp:widelands
Just look at it to see if anything catches your attention that doesn't look right and test the change.
You can also do the code review only and leave the testing to somebody else if you don't have time to test as well.
The purpose of reviews is to help avoid introducing new bugs and to prevent spaghetti code and to make the code easy to read by having a consistent code style.
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/handling-various-fs-errors.