[PATCH xserver 2/2] ranrd: Do not update ConnectionInfo if NULL

Adam Jackson ajax at redhat.com
Thu May 12 16:17:00 UTC 2016


On Thu, 2016-05-12 at 14:51 +0200, Olivier Fourdan wrote:

> @@ -148,7 +148,8 @@ RRScreenSizeNotify(ScreenPtr pScreen)
>  
>      RRTellChanged(pScreen);
>      RRSendConfigNotify(pScreen);
> -    RREditConnectionInfo(pScreen);
> +    if (ConnectionInfo)
> +        RREditConnectionInfo(pScreen);
>  
>      RRPointerScreenConfigured(pScreen);
>      /*

Super nitpicky but I think this reads better if you do the check inside
RREditConnectionInfo.

- ajax


More information about the xorg-devel mailing list