[PATCH] ntdll: Trace RTL_BITMAP api arguments as unsigned when appropriate

Nikolay Sivov nsivov at codeweavers.com
Thu Dec 17 07:06:50 CST 2015


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---

Spotted in traces while testing https://bugs.winehq.org/show_bug.cgi?id=39678

 dlls/ntdll/rtlbitmap.c | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/dlls/ntdll/rtlbitmap.c b/dlls/ntdll/rtlbitmap.c
index 20d6a02..318f6fc 100644
--- a/dlls/ntdll/rtlbitmap.c
+++ b/dlls/ntdll/rtlbitmap.c
@@ -74,7 +74,7 @@ static const signed char NTDLL_mostSignificant[16] = {
  */
 VOID WINAPI RtlInitializeBitMap(PRTL_BITMAP lpBits, PULONG lpBuff, ULONG ulSize)
 {
-  TRACE("(%p,%p,%d)\n", lpBits,lpBuff,ulSize);
+  TRACE("(%p,%p,%u)\n", lpBits,lpBuff,ulSize);
   lpBits->SizeOfBitMap = ulSize;
   lpBits->Buffer = lpBuff;
 }
@@ -130,7 +130,7 @@ VOID WINAPI RtlSetBits(PRTL_BITMAP lpBits, ULONG ulStart, ULONG ulCount)
 {
   LPBYTE lpOut;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulStart, ulCount);
+  TRACE("(%p,%u,%u)\n", lpBits, ulStart, ulCount);
 
   if (!lpBits || !ulCount ||
       ulStart >= lpBits->SizeOfBitMap ||
@@ -191,7 +191,7 @@ VOID WINAPI RtlClearBits(PRTL_BITMAP lpBits, ULONG ulStart, ULONG ulCount)
 {
   LPBYTE lpOut;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulStart, ulCount);
+  TRACE("(%p,%u,%u)\n", lpBits, ulStart, ulCount);
 
   if (!lpBits || !ulCount ||
       ulStart >= lpBits->SizeOfBitMap ||
@@ -254,7 +254,7 @@ BOOLEAN WINAPI RtlAreBitsSet(PCRTL_BITMAP lpBits, ULONG ulStart, ULONG ulCount)
   LPBYTE lpOut;
   ULONG ulRemainder;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulStart, ulCount);
+  TRACE("(%p,%u,%u)\n", lpBits, ulStart, ulCount);
 
   if (!lpBits || !ulCount ||
       ulStart >= lpBits->SizeOfBitMap ||
@@ -327,7 +327,7 @@ BOOLEAN WINAPI RtlAreBitsClear(PCRTL_BITMAP lpBits, ULONG ulStart, ULONG ulCount
   LPBYTE lpOut;
   ULONG ulRemainder;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulStart, ulCount);
+  TRACE("(%p,%u,%u)\n", lpBits, ulStart, ulCount);
 
   if (!lpBits || !ulCount ||
       ulStart >= lpBits->SizeOfBitMap ||
@@ -395,7 +395,7 @@ ULONG WINAPI RtlFindSetBits(PCRTL_BITMAP lpBits, ULONG ulCount, ULONG ulHint)
 {
   ULONG ulPos, ulEnd;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulCount, ulHint);
+  TRACE("(%p,%u,%u)\n", lpBits, ulCount, ulHint);
 
   if (!lpBits || !ulCount || ulCount > lpBits->SizeOfBitMap)
     return ~0U;
@@ -442,7 +442,7 @@ ULONG WINAPI RtlFindClearBits(PCRTL_BITMAP lpBits, ULONG ulCount, ULONG ulHint)
 {
   ULONG ulPos, ulEnd;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulCount, ulHint);
+  TRACE("(%p,%u,%u)\n", lpBits, ulCount, ulHint);
 
   if (!lpBits || !ulCount || ulCount > lpBits->SizeOfBitMap)
     return ~0U;
@@ -489,7 +489,7 @@ ULONG WINAPI RtlFindSetBitsAndClear(PRTL_BITMAP lpBits, ULONG ulCount, ULONG ulH
 {
   ULONG ulPos;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulCount, ulHint);
+  TRACE("(%p,%u,%u)\n", lpBits, ulCount, ulHint);
 
   ulPos = RtlFindSetBits(lpBits, ulCount, ulHint);
   if (ulPos != ~0U)
@@ -514,7 +514,7 @@ ULONG WINAPI RtlFindClearBitsAndSet(PRTL_BITMAP lpBits, ULONG ulCount, ULONG ulH
 {
   ULONG ulPos;
 
-  TRACE("(%p,%d,%d)\n", lpBits, ulCount, ulHint);
+  TRACE("(%p,%u,%u)\n", lpBits, ulCount, ulHint);
 
   ulPos = RtlFindClearBits(lpBits, ulCount, ulHint);
   if (ulPos != ~0U)
@@ -870,7 +870,7 @@ ULONG WINAPI RtlFindNextForwardRunSet(PCRTL_BITMAP lpBits, ULONG ulStart, PULONG
 {
   ULONG ulSize = 0;
 
-  TRACE("(%p,%d,%p)\n", lpBits, ulStart, lpPos);
+  TRACE("(%p,%u,%p)\n", lpBits, ulStart, lpPos);
 
   if (lpBits && ulStart < lpBits->SizeOfBitMap && lpPos)
     *lpPos = NTDLL_FindSetRun(lpBits, ulStart, &ulSize);
@@ -897,7 +897,7 @@ ULONG WINAPI RtlFindNextForwardRunClear(PCRTL_BITMAP lpBits, ULONG ulStart, PULO
 {
   ULONG ulSize = 0;
 
-  TRACE("(%p,%d,%p)\n", lpBits, ulStart, lpPos);
+  TRACE("(%p,%u,%p)\n", lpBits, ulStart, lpPos);
 
   if (lpBits && ulStart < lpBits->SizeOfBitMap && lpPos)
     *lpPos = NTDLL_FindClearRun(lpBits, ulStart, &ulSize);
@@ -922,7 +922,7 @@ ULONG WINAPI RtlFindNextForwardRunClear(PCRTL_BITMAP lpBits, ULONG ulStart, PULO
  */
 ULONG WINAPI RtlFindLastBackwardRunSet(PCRTL_BITMAP lpBits, ULONG ulStart, PULONG lpPos)
 {
-  FIXME("(%p,%d,%p)-stub!\n", lpBits, ulStart, lpPos);
+  FIXME("(%p,%u,%p)-stub!\n", lpBits, ulStart, lpPos);
   return 0;
 }
 
@@ -943,7 +943,7 @@ ULONG WINAPI RtlFindLastBackwardRunSet(PCRTL_BITMAP lpBits, ULONG ulStart, PULON
  */
 ULONG WINAPI RtlFindLastBackwardRunClear(PCRTL_BITMAP lpBits, ULONG ulStart, PULONG lpPos)
 {
-  FIXME("(%p,%d,%p)-stub!\n", lpBits, ulStart, lpPos);
+  FIXME("(%p,%u,%p)-stub!\n", lpBits, ulStart, lpPos);
   return 0;
 }
 
@@ -1021,7 +1021,7 @@ static ULONG NTDLL_FindRuns(PCRTL_BITMAP lpBits, PRTL_BITMAP_RUN lpSeries,
 ULONG WINAPI RtlFindSetRuns(PCRTL_BITMAP lpBits, PRTL_BITMAP_RUN lpSeries,
                             ULONG ulCount, BOOLEAN bLongest)
 {
-  TRACE("(%p,%p,%d,%d)\n", lpBits, lpSeries, ulCount, bLongest);
+  TRACE("(%p,%p,%u,%d)\n", lpBits, lpSeries, ulCount, bLongest);
 
   return NTDLL_FindRuns(lpBits, lpSeries, ulCount, bLongest, NTDLL_FindSetRun);
 }
@@ -1043,7 +1043,7 @@ ULONG WINAPI RtlFindSetRuns(PCRTL_BITMAP lpBits, PRTL_BITMAP_RUN lpSeries,
 ULONG WINAPI RtlFindClearRuns(PCRTL_BITMAP lpBits, PRTL_BITMAP_RUN lpSeries,
                               ULONG ulCount, BOOLEAN bLongest)
 {
-  TRACE("(%p,%p,%d,%d)\n", lpBits, lpSeries, ulCount, bLongest);
+  TRACE("(%p,%p,%u,%d)\n", lpBits, lpSeries, ulCount, bLongest);
 
   return NTDLL_FindRuns(lpBits, lpSeries, ulCount, bLongest, NTDLL_FindClearRun);
 }
-- 
2.6.4




More information about the wine-patches mailing list