From: Keith W. <ke...@va...> - 2001-03-22 17:32:29
|
Gareth Hughes wrote: > > Keith Whitwell wrote: > > > > Gareth, > > > > You only need to pass the reset onto swrast if you are going to draw software > > lines. The current driver only does this when rmesa->Fallback is non-zero. > > Thus I'd suggest: > > > > - Swap between radeonResetLineStipple and _swrast_ResetLineStipple in > > radeonFallback() > > - Call _swrast_ResetLineStipple() once from radeonFallback(), when it is being > > installed. This may not be necessary as fallbacks should occur outside > > begin/end objects on the radeon, so stipple should always be reset. I'd just > > do it anyway. > > Thanks, Keith. I hadn't put much thought into it, I just left that as a > reminder to come back to it later. And when I do, I'll do what you > describe... How certain are you that we won't need any of the POINT_FALLBACK, LINE_FALLBACK, etc code again? I've just cut out a bunch of dead stuff in the latest driver & can add those little changes & commit, if you like. Keith |