[stringtemplate-interest] patch

Roman Odaisky roma at qwertty.com
Tue Nov 11 06:59:32 PST 2008


> Maybe it’s an “if something:” in a place where “if something is not None:”
> should have been used.

Attached patch fixes it.

In getObjectProperty I have replaced “not propertyName” with an explicit 
comparison against "" and None. In ActionEvaluator.attribute I only check for 
None, because str(None) is equal to an undesirable "None", and "" will be 
handled by subsequent getObjectProperty invocation.

-- 
HTH
Roman.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zero.diff
Type: text/x-diff
Size: 1069 bytes
Desc: Patch that treats 0 as non-null
Url : http://www.antlr.org:8080/pipermail/stringtemplate-interest/attachments/20081111/30cc1199/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2286 bytes
Desc: not available
Url : http://www.antlr.org:8080/pipermail/stringtemplate-interest/attachments/20081111/30cc1199/attachment-0001.bin 


More information about the stringtemplate-interest mailing list