Topic Some "issues" with the steering
This board is threaded (i.e. has a tree structure). Please use the Reply button of the specific post you are referring to, not just any random button. If you want to reply to the topic in general, use the Post button near the top and bottom of the page.
In Response to Marky
>But please never comment out code (in the main repository) to deactivate it. If you want to remove code and maybe later re-activate it, then that is what we have revision control for.
Yes, usually I am against deactivating code that way, because the usual situation is that you encounter a line in comments and you have no idea whether it is a temporary change that someone forgot activating again or what its purpose was.
In this case I thought that someone would unlikely look for a change in digging in the shovel, so I decided against simply replacing the code and instead added that comment and conditions when the change should be reverted.
Additionally, you have the confirmation (in the history) in the future if we decided keeping that change or not.p
Powered by mwForum 2.29.7 © 1999-2015 Markus Wichitill