[Freeciv-Dev] [patch #5918] Sanity check boolean effects

Update of patch #5918 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #5918] Sanity check boolean effects

Update of patch #5918 (project freeciv): Status:None => Ready For Test ___ Follow-up Comment #6: Patch to fix the erronous check (file #24105)

[Freeciv-Dev] [patch #5918] Sanity check boolean effects

Update of patch #5918 (project freeciv): Status:Done => None Open/Closed: Closed => Open ___ Reply to this item at:

[Freeciv-Dev] [patch #5918] Sanity check boolean effects

Follow-up Comment #5, patch #5918 (project freeciv): Yes, I guess I've wanted to be extras safe and meant "<= 0" there but got it wrong way. The log message tells what it tries to check against, and that's not true with the current code. ___

[Freeciv-Dev] [patch #5918] Sanity check boolean effects

Follow-up Comment #4, patch #5918 (project freeciv): if (effect_cumulative_min(boolean_effects[i], NULL) < 0 && effect_cumulative_max(boolean_effects[i], NULL) >= 0) { Shouldn't the second test be "=="? This disables the following construct, which I think should be legal (if unnece

[Freeciv-Dev] [patch #5918] Sanity check boolean effects

Update of patch #5918 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

[Freeciv-Dev] [patch #5918] Sanity check boolean effects

URL: Summary: Sanity check boolean effects Project: Freeciv Submitted by: cazfi Submitted on: Sun 08 Mar 2015 02:53:25 AM EET Category: general Priority: 5 - Normal