>>> ============================================================================== >>> >>> --- >>> ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Loop.java >>> (original) >>> +++ >>> ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Loop.java >>> Fri Jan 15 08:14:39 2010 >>> @@ -72,8 +72,8 @@ >>> return false; >>> } >>> >>> - if (count < 1) { >>> - Debug.logWarning("Count is less than one, not doing >>> anything: " + rawString(), module); >>> + if (count < 0) { >>> + Debug.logWarning("Count is less than zero, not doing >>> anything: " + rawString(), module); >>> return true; >>> } >> >> Technically, this is a poor wording; negative counts are more an >> error, and probably shouldn't be happening. > > I like the idea of making things blow up if there is a programming > error. It forces the programmer to fix it.
Cool then. I'll get right on integrating a dynamite library.