agl / xmpp-client

An XMPP client with OTR support
BSD 3-Clause "New" or "Revised" License
365 stars 71 forks source link

Unclear yes/no logic in config prompts #98

Closed mmcco closed 8 years ago

mmcco commented 8 years ago

For the question:

Enable debug logging to /tmp/xmpp-client-debug.log?

(and probably others) there is no indication of what is excepted as a yes or a no. I defaulted to "y", which is usually the safest bet. However, only "yes" is accepted as the affirmative and everything else is assumed to be a no.

My preference would be to accept "y", "yes", and "Yes" as affirmative and "n", "no", and "No" as negative. Everything else would cause the program to reply "not understood" and present the prompt again. You could also add "(yes/no)" to the end of the prompt.

Bikeshedding to be sure, and this write-up became longer than I hoped, but I think it's worth fixing.