You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by gs...@apache.org on 2011/04/08 11:34:20 UTC

svn commit: r1090170 - /qpid/trunk/qpid/tools/src/py/qpid-tool

Author: gsim
Date: Fri Apr  8 09:34:20 2011
New Revision: 1090170

URL: http://svn.apache.org/viewvc?rev=1090170&view=rev
Log:
QPID-3190: always try to eval() quoted tokens

Modified:
    qpid/trunk/qpid/tools/src/py/qpid-tool

Modified: qpid/trunk/qpid/tools/src/py/qpid-tool
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/tools/src/py/qpid-tool?rev=1090170&r1=1090169&r2=1090170&view=diff
==============================================================================
--- qpid/trunk/qpid/tools/src/py/qpid-tool (original)
+++ qpid/trunk/qpid/tools/src/py/qpid-tool Fri Apr  8 09:34:20 2011
@@ -267,10 +267,10 @@ class QmfData(Console):
       ##
       ## TODO: use a regex for this instead of this convoluted logic,
       ## or even consider passing all args through eval() [which would
-      ## be a minor change to the nterface as string args would then
+      ## be a minor change to the interface as string args would then
       ## always need to be quoted as strings within a map/list would
       ## now]
-      if arg[0] == '{' or arg[0] == '[' or arg == "True" or arg == "False" or \
+      if arg[0] == '{' or arg[0] == '[' or arg[0] == '"' or arg[0] == '\'' or arg == "True" or arg == "False" or \
             ((arg.count('.') < 2 and (arg.count('-') == 0 or \
             (arg.count('-') == 1 and  arg[0] == '-')) and \
             arg.replace('.','').replace('-','').isdigit())):



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org