Typo in recent xf86-video-ati commit (ATOM: Upstream parser updates)
Oldřich Jedlička
oldium.pro at seznam.cz
Wed Jan 20 09:42:28 PST 2010
Hi Alex,
On Wednesday 20 of January 2010 at 15:20:14, Alex Deucher wrote:
> 2010/1/20 Oldřich Jedlička <oldium.pro at seznam.cz>:
> > Hi Alex,
> >
> > I've just found that there are new commits in xf86-video-ati and one is
> > suspicious to me - ATOM: Upstream parser updates. It contains lines:
> >
> >
> > pParserTempData->DestData32=GetDestination[pParserTempData->ParametersTy
> > pe.Destination](pParserTempData);
> > pParserTempData->SourceData32=GetParametersDirect(pParserTempData); -
> > pParserTempData->Index=GetParametersDirect(pParserTempData); +
> > pParserTempData->Index=GetSource[pParserTempData->ParametersType.Source
> > ](pParserTempData);
> >
> > Shouldn't the Index change go to SourceData32? It uses the same
> > pattern as DestData32?
>
> No, the code is correct as is. The mask op takes two sources: an AND
> mask and an OR mask. The AND mask is always inline, while the OR mask
> is now a full param.
Thanks for clarification.
Oldřich.
> Alex
> _______________________________________________
> xorg-devel mailing list
> xorg-devel at lists.x.org
> http://lists.x.org/mailman/listinfo/xorg-devel
More information about the xorg-devel
mailing list