On Wed, Jun 19, 2013 at 11:54 AM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> Hi Lucas,
>
>>>>>> Having no description doesn't mean it's not an available command. So add
>>>>>> command to the output of "help" command, even if it has no description.
>>>>>> ---
>>>>>> client/main.c | 5 ++---
>>>>>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>>>>>
>>>>>> diff --git a/client/main.c b/client/main.c
>>>>>> index b0a66d8..54af807 100644
>>>>>> --- a/client/main.c
>>>>>> +++ b/client/main.c
>>>>>> @@ -1182,10 +1182,9 @@ static void rl_handler(char *input)
>>>>>> printf("Available commands:\n");
>>>>>>
>>>>>> for (i = 0; cmd_table[i].cmd; i++) {
>>>>>> - if (cmd_table[i].desc)
>>>>>> - printf("\t%s %s\t%s\n", cmd_table[i].cmd,
>>>>>> + printf("\t%s %s\t%s\n", cmd_table[i].cmd,
>>>>>> cmd_table[i].arg ? : " ",
>>>>>> - cmd_table[i].desc);
>>>>>> + cmd_table[i].desc ? : "");
>>>>>
>>>>>
>>>>> if I remember this correctly, then I was doing it to be able to hide
>>>>> commands from the help output. Mainly commands that are on purpose
>>>>> duplicates.
>>>>
>>>> So the right fix is to add the missing descriptions then? Lucas, can you
>>>> provide an updated patch that does this?
>>>
>>> actually I wanted to hide these commands on purpose.
>>
>> So you can't pair a device starting from remote using bluetoothctl on
>> purpose? Or is adding a description to at least "default-agent" ok?
>
> is this for the "default-agent" command? I forgot to add a description for that one?
Yes. I went with "show even if not description is available" because
it doesn't do any harm. I can add a description to default-agent
though
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
