Re: [Warzone2100-project] [Warzone2100-commits] [Warzone2100/warzone2100] 3e48f5: Hideous indeed.
On 07/27/2015 04:14 AM, vexed wrote: On Sun, Jul 26, 2015 at 2:32 PM, Kreuvf wrote: On 07/26/2015 04:27 AM, vexed wrote: Log Message: --- Hideous indeed. Bad things happen when player 6. Is this a case of https://xkcd.com/1296/? Please be more descriptive in your commit messages, there are people who actually read them :) Best regards - Kreuvf That *was* being descriptive though... if player's value was less than 6, then, we spam errors. You could also look at the commit in question, and see that all that was removed was, - 6, then think, yeah, I understand. Yes, I _could_, but I rather look at commits, because the commit message sparked my interest, not because it puzzled me. But, you mean, you rather me type something like... We don't want negative players, WZ has enough of them as it is.? ;) Making it say Fix issue where player value could be negative is basically the same thing. I prefer Fix issue where player value could be negative. Best regards - Kreuvf signature.asc Description: OpenPGP digital signature -- ___ Warzone2100-project mailing list Warzone2100-project@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/warzone2100-project
Re: [Warzone2100-project] [Warzone2100-commits] [Warzone2100/warzone2100] 3e48f5: Hideous indeed.
On Tue, Jul 28, 2015 at 12:13:37AM +0200, Kreuvf wrote: Yes, I _could_, but I rather look at commits, because the commit message sparked my interest, not because it puzzled me. But, you mean, you rather me type something like... We don't want negative players, WZ has enough of them as it is.? ;) Making it say Fix issue where player value could be negative is basically the same thing. I prefer Fix issue where player value could be negative. I'm with Kreuf here. The summary line should describe what the commit does. Clever stuff can go in the commit msg body. Having to look at the code to decipher the meaning is Fail. S. -- Stephen Swaney sswa...@centurytel.net 231-313-2492skype: sswaney (GMT-4) -- ___ Warzone2100-project mailing list Warzone2100-project@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/warzone2100-project
Re: [Warzone2100-project] [Warzone2100-commits] [Warzone2100/warzone2100] 3e48f5: Hideous indeed.
On 07/26/2015 04:27 AM, vexed wrote: Branch: refs/heads/master Home: https://github.com/Warzone2100/warzone2100 Commit: 3e48f5526d1d2b2fe7ffdfb5f0e606214efb11fa https://github.com/Warzone2100/warzone2100/commit/3e48f5526d1d2b2fe7ffdfb5f0e606214efb11fa Author: vexed ve...@wz2100.net Date: 2015-07-25 (Sat, 25 Jul 2015) Changed paths: M src/component.cpp Log Message: --- Hideous indeed. Bad things happen when player 6. Is this a case of https://xkcd.com/1296/? Please be more descriptive in your commit messages, there are people who actually read them :) Best regards - Kreuvf signature.asc Description: OpenPGP digital signature -- ___ Warzone2100-project mailing list Warzone2100-project@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/warzone2100-project