Re: [PATCH, v17] Add support for setting disassembler-options in GDB for POWER, ARM and S390

[Pedro Alves <palves at redhat dot com>]

On 02/24/2017 05:13 PM, Peter Bergner wrote:

>   if (valid_options->description != NULL)
>     {
>       ...
>     }
>   else
>     {
>       size_t i;
>       fprintf_filtered (file, "  ");
>       for (i = 0; valid_options->name[i] != NULL; i++)
>         {
>           fprintf_filtered (file, "%s", valid_options->name[i]);
>           if (valid_options->name[i+1] != NULL)
>             fprintf_filtered (file, ", ");
>           wrap_here ("  ");
>         }
>       fprintf_filtered (file, "\n");
>     }
> 

Yup, that looks fine.  (with spaces around the '+' operator).

Thanks,
Pedro Alves