xserver: Branch 'master'

Daniel Stone daniel at fooishbar.org
Tue Mar 11 00:20:13 PDT 2008


Hi,
Thanks for actually taking on XKB ...

On Mon, Mar 10, 2008 at 09:29:27PM -0700, Keith Packard wrote:
> @@ -1796,6 +1796,7 @@ XkbCopyKeymap(XkbDescPtr src, XkbDescPtr dst, Bool sendNotifies)
>                    dsection = dst->geom->sections;
>                   i < src->geom->num_sections;
>                   i++, ssection++, dsection++) {
> +		*dsection = *ssection;

Please don't introduce tab breakage into an otherwise consistent file.

> @@ -1830,6 +1831,7 @@ XkbCopyKeymap(XkbDescPtr src, XkbDescPtr dst, Bool sendNotifies)
>                      dsection->doodads = NULL;
>                  }
>  
> +                dsection->sz_doodads = ssection->num_doodads;

If sz_doodads is set, you should be setting num_doodads for consistency,
unless this is already covered by *dsection = *ssection.

> @@ -1850,8 +1852,9 @@ XkbCopyKeymap(XkbDescPtr src, XkbDescPtr dst, Bool sendNotifies)
>                      }
>                      ddoodad->any.type = sdoodad->any.type;
>                  }
> -                dsection->num_doodads = ssection->num_doodads;
> -                dsection->sz_doodads = ssection->num_doodads;
> +		dsection->overlays = NULL;
> +		dsection->sz_overlays = 0;
> +		dsection->num_overlays = 0;

Again, indentation failure.

Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg/attachments/20080311/5d3f0e3b/attachment.pgp>


More information about the xorg mailing list