-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/701/#review896
-----------------------------------------------------------
cli/lmi/shell/LMIUtil.py
<
http://reviewboard-openlmi.rhcloud.com/r/701/#comment480>
Shouldn't it be like:
... cast[t.lower()] if t.lower() in cast ...
This way it checks if lowercased t is key in the dict but it gets non-lowercased key.
- Radek Novacek
On Aug. 9, 2013, 9:53 a.m., Peter Hatina wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/701/
-----------------------------------------------------------
(Updated Aug. 9, 2013, 9:53 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description
-------
When retrieving constant values from either LMIClass or LMIMethod, cast them to
python/lmishell native types before returning.
Diffs
-----
cli/lmi/shell/LMIConstantValues.py 6e564ac413f3fbcb0afc3a8406b55d7701a3ab71
cli/lmi/shell/LMIUtil.py 5954b9650f358e53fb75e08d9aac30365be8964f
Diff:
http://reviewboard-openlmi.rhcloud.com/r/701/diff/
Testing
-------
Thanks,
Peter Hatina