Make -axis shell argument not case sensitive

Issue #164 resolved
Eliot Phillips created an issue

Using the mirror command's -axis argument requires a capital letter X or Y and fails on x or y.

Comments (6)

  1. Juan Pablo Caram repo owner

    @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.

  2. Kamil Sopko

    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.

  3. Kamil Sopko

    but all option names are for now lowercase do we have to mess with this? same situation is with command names

  4. Juan Pablo Caram 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.

  5. Juan Pablo Caram repo owner

    I just tested this and the problem is not there. axis = kwa['axis'].upper() has been there for quite some time now.

  6. Log in to comment