Changeset 587 for trunk/src/ddraw/OS2SURFACE.CPP

Timestamp:

Aug 19, 1999, 8:08:48 PM (26 years ago)

Author:

hugh

Message:

Implemented new colorconversion routine and fixed bug in fastblt function.
Changed the palettehandling code

File:

• trunk/src/ddraw/OS2SURFACE.CPP (modified) (22 diffs)

trunk/src/ddraw/OS2SURFACE.CPP

@@ -6 +6 @@
                   ( (DWORD)(BYTE)(ch2) << 16 ) | ( (DWORD)(BYTE)(ch3) << 24 ) )
 #include <fourcc.h>
+#define INITGUID
+#include "os2ddraw.h"
+#include "os2clipper.h"
+#include "os2palette.h"
 #include "os2surface.h"
 #include "rectangle.h"
@@ -154 +158 @@
   WriteLog("\n");
 }
+
+static void _dump_DDSCAPS2(DWORD flagmask) {
+  int  i;
+  const struct {
+    DWORD  mask;
+    char  *name;
+  } flags[] = {
+#define FE(x) { x, #x},
+    FE(DDSCAPS2_HARDWAREDEINTERLACE)
+    FE(DDSCAPS2_HINTANTIALIASING)
+    FE(DDSCAPS2_HINTDYNAMIC)
+    FE(DDSCAPS2_HINTSTATIC)
+    FE(DDSCAPS2_OPAQUE)
+    FE(DDSCAPS2_TEXTUREMANAGE)
+  };
+  for (i=0;i<sizeof(flags)/sizeof(flags[0]);i++)
+    if (flags[i].mask & flagmask)
+      WriteLog("%s ",flags[i].name);
+  WriteLog("\n");
+}
+
 
 static void _dump_DDSD(DWORD flagmask) {
@@ -553 +578 @@
       height = DDSurfaceDesc.dwHeight;
 
+      lpDraw->pPrimSurf = this;
+
       lastError = DD_OK;
       return;
@@ -591 +618 @@
       {
         WriteLog(" Use Screen Format :");
-        dwBpp = dwBytesPPDive << 3; // No use Screenformat
+        dwBpp = lpDraw->GetScreenBpp(); // No use Screenformat
         lpDDSurfaceDesc->ddpfPixelFormat.dwSize = sizeof(DDPIXELFORMAT);
         lpDDSurfaceDesc->ddpfPixelFormat.dwFourCC = (DWORD) lpDraw->GetScreenFourCC();
@@ -740 +767 @@
           dwPitchFB = lpDDSurfaceDesc->dwWidth * (dwBpp<8?1:dwBpp/8);
           dwPitchFB = (dwPitchFB +7) & ~7;  // Align on QWords
-
           DDSurfaceDesc.lPitch    = dwPitchFB;
           lpDDSurfaceDesc->lPitch = dwPitchFB;
@@ -764 +790 @@
           pFrameBuffer = (char*)(((int)pFBreal + 7) & ~7); // align to QWORD
 
-          WriteLog( " Framebuffer @ %08X QWAligned @ %08X with a Pitch of %d",
+          WriteLog( " Framebuffer @ %08X QWAligned @ %08X with a Pitch of %d\n",
                     pFBreal, pFrameBuffer, dwPitchFB);
 
@@ -780 +806 @@
           {
             WriteLog(" Alloc CCBuf ");
+            dwPitchDB = (lpDDSurfaceDesc->dwWidth * (lpDraw->dCaps.ulDepth/8) +7) & ~7;
             if(Mainchain)
             {
@@ -790 +817 @@
                                     lpDDSurfaceDesc->dwWidth,
                                     lpDDSurfaceDesc->dwHeight,
-                                    (lpDDSurfaceDesc->dwWidth * (lpDraw->dCaps.ulDepth/8) +7) & ~7,
+                                    dwPitchDB,
                                     (PBYTE)pDiveBuffer);
             }
             else
             {
-              WriteLog("with malloc");
+              WriteLog( "with malloc (%dx%d) Pitch %d ",
+                        lpDDSurfaceDesc->dwHeight,
+                        lpDDSurfaceDesc->dwWidth,
+                        dwPitchDB);
               // No so we must create the Divebuffer to do the colortranslation
               // and blit to the real framebuffer on Unlock to do color conversion
               pDiveBuffer = (char*)malloc( lpDDSurfaceDesc->dwHeight *
-                                   ((lpDDSurfaceDesc->dwWidth * (lpDraw->dCaps.ulDepth/8) +7) & ~7) );
+                                           dwPitchDB);
             }
+            WriteLog( "  @ %08X\n", pDiveBuffer);
           }
 
@@ -1154 +1185 @@
 void OS2IDirectDrawSurface::ColorConversion(LPRECT lpRect)
 {
+  char *pSrc,*pDst,*pSLine,*pDLine;
+  DWORD CCwidth,CCheight,x,y;
+  DWORD *pPal24;
+  WORD  *pPal16;
+
+  if(NULL== lpDraw->pPrimSurf)
+  {
+    WriteLog("No Primary Surface! => No CC\n");
+    return;
+  }
+
+  if(NULL== ((OS2IDirectDrawSurface*)lpDraw->pPrimSurf)->lpPalette)
+  {
+    WriteLog("No DDrawpalette defined!=> No CC\n");
+    return;
+  }
+
+  if(NULL==lpRect)
+  {
+    pSrc = pFrameBuffer;
+    pDst = pDiveBuffer;
+    CCwidth  = width;
+    CCheight = height;
+  }
+  else
+  {
+    // ToDo: Check why the top/bottom values come in swaped here
+    // for now simply reverse them with the following 3 lines
+    y = lpRect->top;
+    lpRect->top = lpRect->bottom;
+    lpRect->bottom = y;
+    // end of hack
+    pSrc = pFrameBuffer +
+           (lpRect->top * dwPitchFB) +
+           (lpRect->left* (lpDraw->dCaps.ulDepth/8));
+    pDst = pDiveBuffer +
+           (lpRect->top * dwPitchDB) +
+           (lpRect->left* dwBytesPPDive);
+    CCwidth  = lpRect->right - lpRect->left;
+    CCheight = lpRect->bottom - lpRect->top;
+
+  }
+
+  WriteLog( "H: %d W: %d\n SRC @ %08X\n DST @ %08X\n",
+            CCheight, CCwidth, pSrc,pDst);
+
+  pSLine = pSrc;
+  pDLine = pDst;
+
+  switch(lpDraw->dCaps.ulDepth)
+  {
+    case 8:
+      WriteLog(" 8Bit target CC not implemented\n");
+      break;
+    case 15:
+      WriteLog(" 15 Bit not implemented using 16bit might look ugly\n");
+    case 16:
+      if(8==lpDraw->GetScreenBpp())
+      {
+        pPal16 = ((OS2IDirectDrawSurface*)lpDraw->pPrimSurf)->lpPalette->aPal16;
+        WriteLog("8->16Bit CC\n");
+        for(y=0;CCheight;CCheight--,y++)
+        {
+          WriteLog("Converting Line %d\n",y);
+          for(x=0;x<width;x++)
+          {
+            *((WORD*)pDLine) = pPal16[pSLine[x]];
+          }
+          pSLine += dwPitchFB;
+          pDLine += dwPitchDB;
+        }
+      }
+      else
+      {
+        WriteLog("%d ->16Bit Not implemented",lpDraw->GetScreenBpp());
+      }
+      break;
+    case 24:
+      if(8==lpDraw->GetScreenBpp())
+      {
+        WriteLog("8->24Bit CC");
+      }
+      else
+      {
+        WriteLog("%d ->16Bit Not implemented",lpDraw->GetScreenBpp());
+      }
+      break;
+    case 32:
+      if(8==lpDraw->GetScreenBpp())
+      {
+        WriteLog("8->32Bit CC");
+      }
+      else
+      {
+        WriteLog("%d ->16Bit Not implemented",lpDraw->GetScreenBpp());
+      }
+      break;
+    default:
+      #ifdef DEBUG
+        WriteLog( "Unexpected Screen Bitdepth %d\n",
+                  lpDraw->dCaps.ulDepth);
+      #endif
+      break;
+  }
+
+/*
   SETUP_BLITTER sBlt;
   ULONG ulDN1, ulDN2;
@@ -1182 +1319 @@
   sBlt.lDstPosY          = sBlt.ulSrcPosY;
   sBlt.ulNumDstRects     = DIVE_FULLY_VISIBLE;
+
+  WriteLog( "Colorconversion:\n FCC SRC %08X\n FCC DST %08X\n",
+            sBlt.fccSrcColorFormat,
+            sBlt.fccDstColorFormat );
 
   rc = DiveAllocImageBuffer( hDiveCC,
@@ -1190 +1331 @@
                              dwPitchFB,
                              (PBYTE)pFrameBuffer);
-  WriteLog("AllocDiveSrc Buffer rc= %X\n",rc);
+  WriteLog("AllocDiveSrc Buffer rc= 0x%08X\n",rc);
 
   rc = DiveAllocImageBuffer( hDiveCC,
@@ -1199 +1340 @@
                              dwPitchDB,
                              (PBYTE)pDiveBuffer);
-  WriteLog("AllocDiveDst Buffer rc= %Xd\n",rc);
+  WriteLog("AllocDiveDst Buffer rc= 0x%08X\n",rc);
 
   rc = DiveSetupBlitter( hDiveCC,
@@ -1219 +1360 @@
 
   WriteLog("Free dst rc= %X\n",rc);
-
+*/
 }
 //******************************************************************************
@@ -2571 +2712 @@
   BlitWidth  = (SrcRect.right - SrcRect.left) * src->dwBytesPPDive;
 
-  if(dwTrans & DDBLTFAST_NOCOLORKEY)
+  // Remove unsupported wait flag
+  dwTrans &= ~DDBLTFAST_WAIT;
+
+  if(DDBLTFAST_NOCOLORKEY == dwTrans )
   {
     WriteLog("Solid Blit");
@@ -2593 +2737 @@
   else
   {
-    WriteLog("TransBlit");
+    WriteLog("TransBlit\n");
 
     if(dwTrans & DDBLTFAST_SRCCOLORKEY)
     {
+      WriteLog("Trans SRC\n");
       // transparent source
       dwSrcColor = src->DDSurfaceDesc.ddckCKSrcBlt.dwColorSpaceLowValue;
       if(dwTrans & DDBLTFAST_DESTCOLORKEY)
       {
+        WriteLog("And Dest Colorkey");
         dwDestColor = dest->DDSurfaceDesc.ddckCKDestBlt.dwColorSpaceLowValue;
         // Source and dest colorkeying
@@ -2697 +2843 @@
         // This MMX detection should be moved into OS2Draw
         // and into the surface constructor a setup for blitting pointers
-
+        WriteLog("Only Src ColorKey");
         switch(dest->dwBytesPPDive)
         {
@@ -2768 +2914 @@
       if(dwTrans & DDBLTFAST_DESTCOLORKEY)
       {
+        WriteLog("DestColorKey\n");
+
         dwDestColor = dest->DDSurfaceDesc.ddckCKDestBlt.dwColorSpaceLowValue;
         switch(dest->dwBytesPPDive)
@@ -2844 +2992 @@
         }  // End switch
       }
+      else
+      {
+        WriteLog("Unexpected Flags");
+      }
     }
   }
@@ -3349 +3501 @@
 
   lpDDCaps->dwCaps = me->DDSurfaceDesc.ddsCaps.dwCaps;
-
+  #ifdef DEBUG
+    _dump_DDSCAPS(lpDDCaps->dwCaps);
+  #endif
   return(DD_OK);
 }
@@ -3365 +3519 @@
 
   memcpy(lpDDCaps, &(me->DDSurfaceDesc.ddsCaps), sizeof(DDSCAPS2) );
+  #ifdef DEBUG
+    _dump_DDSCAPS(lpDDCaps->dwCaps);
+    _dump_DDSCAPS2(lpDDCaps->dwCaps2);
+
+  #endif
 
   return(DD_OK);
@@ -4098 +4257 @@
       // primary surface => doesn't modify physical palette
       if(me->surfaceType & DDSCAPS_PRIMARYSURFACE)
+      {
         me->lpPalette->SetIsPrimary(FALSE);
-
+      }
       me->lpPalette->Vtbl.Release((IDirectDrawPalette*)me->lpPalette);
       me->lpPalette = NULL;
@@ -4128 +4288 @@
 
   if(me->surfaceType & DDSCAPS_PRIMARYSURFACE)
+  {
     me->lpPalette->SetIsPrimary(TRUE);
-
+  }
   me->lpVtbl->ChangeUniquenessValue(me);