This block in src/savage_dri.c looks suspiciously like cruft. I can not make any sense of that block and its local, unused variable:
1119 unsigned int shift = 0;
1121 if(pSAVAGEDRI->width > 1024)
1122 shift = 1;
1124 pSAVAGEDRI->aperturePitch = psav->ulAperturePitch;
The last time anyone touched this block was commit 54a78b4d252ba06dd520c5d43ad55b31323408ec but this has been like this since as long as git can tell.
Created attachment 10178 [details] [review]
removes unnecessary block and code
I guess it should be just like this.
Fixed in git, thanks!