Make -axis shell argument not case sensitive
Using the mirror
command's -axis
argument requires a capital letter X or Y and fails on x or y.
Comments (6)
-
repo owner -
for name of option it would be possible do it globally in new commands(TclCommand argument handling), but it is not good idea do it for options , for example name of output FILE and maybe name of object should be case sensitive.
if we need it , then we need it implemented it in specific commands.
-
repo owner @sopak Correct, for option names only. Everything else stays case sensitive.
-
but all option names are for now lowercase do we have to mess with this? same situation is with command names
-
repo owner @sopak Oh, you are right. This issue is about the value, not the option name.
If you have implemented this command in the new way I can make the change to fix this issue.
-
repo owner - changed status to resolved
I just tested this and the problem is not there.
axis = kwa['axis'].upper()
has been there for quite some time now. - Log in to comment
@sopak could we make options case insensitive? I was wondering if this was a quick fix that we could do for version 8.5 or deffer it to 9.0.