[PATCH xrandr] Print spaces between XA_ATOM property values

[Peter Hutterer]

On Thu, Feb 28, 2013 at 12:12:01PM -0800, Aaron Plattner wrote:
> Commit b26fd532b3dab222956ea27eef4e41345978b5b2 redid how xrandr prints
> properties.  It neglected to put spaces between the values of XA_ATOM
> properties, so they all run together.  For example,
> 
>     audio: auto
>         supported: force-dvioffautoon
> 
> Fix this by putting a space after each atom name.  In addition, some drivers
> create property values with spaces in them, so put commas between entries to
> disambiguate.  For example,
> 
>     Broadcast RGB: Automatic
>         supported: Automatic, Full, Limited 16:235
> 
> Do the same for properties with multiple valid ranges.
> 
> Signed-off-by: Aaron Plattner <aplattner at nvidia.com>

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
   Peter

> ---
>  xrandr.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/xrandr.c b/xrandr.c
> index 0e3818f..305308c 100644
> --- a/xrandr.c
> +++ b/xrandr.c
> @@ -3560,6 +3560,8 @@ main (int argc, char **argv)
>  			    print_output_property_value (False, 32, actual_type,
>  							 (unsigned char *) &(propinfo->values[k * 2 + 1]));
>  			    printf (")");
> +			    if (k < propinfo->num_values / 2 - 1)
> +				printf (", ");
>  			}
>  			printf ("\n");
>  		    }
> @@ -3570,6 +3572,8 @@ main (int argc, char **argv)
>  			{
>  			    print_output_property_value (False, 32, actual_type,
>  							 (unsigned char *) &(propinfo->values[k]));
> +			    if (k < propinfo->num_values - 1)
> +				printf (", ");
>  			}
>  			printf ("\n");
>  		    }
> -- 
> 1.7.12
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>