http://codereview.appspot.com/74075/diff/1/8
File
java/common/src/test/java/org/apache/shindig/expressions/ExpressionsTest.java
(right):

http://codereview.appspot.com/74075/diff/1/8#newcode174
Line 174: }
On 2009/06/17 22:18:49, levik wrote:
What about the string "0"? What should that do?

The same thing it does in Javascript - coerce to true.

OTOH, the string "false" doesn't coerce to false in JS, so I could be
pushed to either:
  - Make "0" -> false
  - Or make "false" -> true
  - Or just leave things as they are

The current state is a bit inconsistent.

http://codereview.appspot.com/74075

Reply via email to