xserver: Branch 'master'
Keith Packard
keithp at keithp.com
Tue Mar 11 09:07:45 PDT 2008
On Tue, 2008-03-11 at 15:46 +0000, Bill Crawford wrote:
> On 11/03/2008, Daniel Stone <daniel at fooishbar.org> wrote:
>
> > > @@ -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.
>
> We had that conversation on xorg-devel a few weeks ago :) it's
> supposed to be num_doodads there, as the alloc of the memory uses
> num_doodads for the element count.
The version I submitted is correct -- num_doodads is set in the global
structure assignment above (*dsection = *ssection).
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg/attachments/20080311/2d82cfd4/attachment.pgp>
More information about the xorg
mailing list