Removes unused (big big) array. Adds ifdef for debug stuff.
authorPetr Sumbera <sumbera@volny.cz>
Wed, 22 Jul 2015 15:51:09 +0000 (08:51 -0700)
committerPetr Sumbera <sumbera@volny.cz>
Wed, 22 Jul 2015 15:51:09 +0000 (08:51 -0700)
server/shadow/shadow_capture.c

index f7035bb..42163f8 100644 (file)
@@ -80,9 +80,6 @@ int shadow_capture_align_clip_rect(RECTANGLE_16* rect, RECTANGLE_16* clip)
        return 1;
 }
 
-/* this is too big to consume whole stack on some systems */
-static BOOL grid[1024][1024];
-
 int shadow_capture_compare(BYTE* pData1, int nStep1, int nWidth, int nHeight, BYTE* pData2, int nStep2, RECTANGLE_16* rect)
 {
        BOOL equal;
@@ -93,13 +90,16 @@ int shadow_capture_compare(BYTE* pData1, int nStep1, int nWidth, int nHeight, BY
        int l, t, r, b;
        BYTE *p1, *p2;
        BOOL rows[1024];
+#ifdef WITH_DEBUG_SHADOW_CAPTURE
        BOOL cols[1024];
+#endif
 
        allEqual = TRUE;
        ZeroMemory(rect, sizeof(RECTANGLE_16));
        FillMemory(rows, sizeof(rows), 0xFF);
+#ifdef WITH_DEBUG_SHADOW_CAPTURE
        FillMemory(cols, sizeof(cols), 0xFF);
-       FillMemory(grid, sizeof(grid), 0xFF);
+#endif
 
        nrow = (nHeight + 15) / 16;
        ncol = (nWidth + 15) / 16;
@@ -143,9 +143,10 @@ int shadow_capture_compare(BYTE* pData1, int nStep1, int nWidth, int nHeight, BY
 
                        if (!equal)
                        {
-                               grid[ty][tx] = FALSE;
                                rows[ty] = FALSE;
+#ifdef WITH_DEBUG_SHADOW_CAPTURE
                                cols[tx] = FALSE;
+#endif
 
                                if (l > tx)
                                        l = tx;
@@ -181,40 +182,39 @@ int shadow_capture_compare(BYTE* pData1, int nStep1, int nWidth, int nHeight, BY
        if (rect->bottom > nHeight)
                rect->bottom = nHeight;
 
-       if (0)
+#ifdef WITH_DEBUG_SHADOW_CAPTURE
+       char *col_str = calloc(ncol + 1, sizeof(char));
+       if (!col_str)
        {
-               char *col_str = calloc(ncol + 1, sizeof(char));
-               if (!col_str)
-               {
-                       WLog_ERR(TAG, "calloc failed!");
-                       return 1;
-               }
+               WLog_ERR(TAG, "calloc failed!");
+               return 1;
+       }
 
-               for (tx = 0; tx < ncol; tx++)
-                       sprintf(&col_str[tx], "-");
-               WLog_INFO(TAG, "%s", col_str);
+       for (tx = 0; tx < ncol; tx++)
+               sprintf(&col_str[tx], "-");
+       WLog_INFO(TAG, "%s", col_str);
 
-               for (tx = 0; tx < ncol; tx++)
-                       sprintf(&col_str[tx], "%c", cols[tx] ? 'O' : 'X');
-               WLog_INFO(TAG, "%s", col_str);
+       for (tx = 0; tx < ncol; tx++)
+               sprintf(&col_str[tx], "%c", cols[tx] ? 'O' : 'X');
+       WLog_INFO(TAG, "%s", col_str);
+
+       for (tx = 0; tx < ncol; tx++)
+               sprintf(&col_str[tx], "-");
+       WLog_INFO(TAG, "%s", col_str);
 
+       for (ty = 0; ty < nrow; ty++)
+       {
                for (tx = 0; tx < ncol; tx++)
-                       sprintf(&col_str[tx], "-");
+                       sprintf(&col_str[tx], "%c", cols[tx] ? 'O' : 'X');
                WLog_INFO(TAG, "%s", col_str);
-
-               for (ty = 0; ty < nrow; ty++)
-               {
-                       for (tx = 0; tx < ncol; tx++)
-                               sprintf(&col_str[tx], "%c", cols[tx] ? 'O' : 'X');
-                       WLog_INFO(TAG, "%s", col_str);
-                       WLog_INFO(TAG, "|%s|", rows[ty] ? "O" : "X");
-               }
-
-               WLog_INFO(TAG, "left: %d top: %d right: %d bottom: %d ncol: %d nrow: %d",
-                               l, t, r, b, ncol, nrow);
-               free(col_str);
+               WLog_INFO(TAG, "|%s|", rows[ty] ? "O" : "X");
        }
 
+       WLog_INFO(TAG, "left: %d top: %d right: %d bottom: %d ncol: %d nrow: %d",
+                       l, t, r, b, ncol, nrow);
+       free(col_str);
+#endif
+
        return 1;
 }