Re: Changing movement
Posted: 2010-08-19, 00:49
Sorry, but it is very difficult to make sense of that portion of code:
With a patch I'd not only be able to see the code exactly the same as you, I'd also see the whole picture, and I'd even be able to try and run it myself, and this besides all the other advantages mentioned in the above article.
Btw., with TortoiseSVN creating a patch is not more than right-clicking to open the context-menu and choosing where the patch should be saved.
- For a start, indentation is wrong, e.g. the curly brackets in
Code: Select all
if (...) { ...; }
- It's very difficult to see where the code begins and where it ends, what variables you declare, initialize and use where, etc.
With a patch I'd not only be able to see the code exactly the same as you, I'd also see the whole picture, and I'd even be able to try and run it myself, and this besides all the other advantages mentioned in the above article.
Btw., with TortoiseSVN creating a patch is not more than right-clicking to open the context-menu and choosing where the patch should be saved.