Confusing the option name by the case-sensitive letter

[iamwonseok]

I think the option name may confused at the letter 'd' or 'D'.

In case of 'd', it uses to specify the directory name, while 'D' uses to configure a certain level of depth.

As I know, many programmers prefer to use shortcut key for simplicity. However, uftrace provides two features with the same letter, even if they can be distinguished by case-sensitive. It would be confusing to users compared to using different letters.

For example, du command that is widely used in linux to get disk usage, it uses 'd' for level of depth. In addition, most of application uses 'o' letter to specify the directory name for output data.

So, I think uftrace needs to modify the name of option to make it clearer. It would be more familiar with users.

What do you think about?

[namhyung]

I thought about it. But the -d/--data option is also used for replay, report, graph and dump. I guess -i/--input is more appropriate in this case, but I'd like to have a single option for both cases.

[iamwonseok]

I see. As your mention, I think that '-d/--data' option is suitable for those cases. However, I am still thinking that using different characters give to user more intuitive.

If so, what do you think of changing the depth from 'D' to 'L' or 'l'? L or l means the depth-level.

[namhyung]

It'll limit the max number of short options to 26, right? And I don't understand why it needs to avoid same character. Do you want to make the short options case-insensitive? I think 'D' is the base one for depth filter and I somehow managed most of filters to have upper cases. And it'd be bad to change an existing option to have a different meaning.

[iamwonseok]

Sure, if short options are used without case-sensitive, the limit is 26. But, that's not what I meant to say. I don't want to make the short options case sensitive. I just asked what you think about changing from D to l or L, if it is possible. Now, I understood what you intention is and I agree with your opinion.

Thank you for your answer. I am sorry to make you confused.

[namhyung]

No need to be sorry. I was very glad to hear your opinion. Getting feedback from user is always good. Please keep using uftrace and post issues if you find anything inconvenient or unreasonable (hopefully with patches!). Thanks :)