Code review comment for lp:~majcherlk/widelands/bug-1243700

Revision history for this message
SirVer (sirver) wrote :

thanks, I understood that while reading this merge request because I had the context. What I meant is that you should add this documentation to the code, so that future readers of the code still understood your thinking.

Sorry for not being clear - generally when somebody asks for clarification in a code review, it is always best to answer in the code, not the comments.

review: Needs Fixing

« Back to merge proposal