https://git.reactos.org/?p=reactos.git;a=commitdiff;h=3ca1ac639cb597550b5355...
commit 3ca1ac639cb597550b5355827d29edc485f696bb Author: Serge Gautherie 32623169+SergeGautherie@users.noreply.github.com AuthorDate: Wed May 30 14:18:23 2018 +0200 Commit: Hermès BÉLUSCA - MAÏTO hermes.belusca-maito@reactos.org CommitDate: Wed May 30 14:18:23 2018 +0200
[REACTOS] Improve/Fix some "printf" formats (#555)
The smbios.c one was initially spotted by "Alahndro" on CORE-6328. --- dll/win32/shell32/CDefView.cpp | 8 ++++---- ntoskrnl/wmi/smbios.c | 2 +- win32ss/user/ntuser/cursoricon.c | 8 ++++---- 3 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/dll/win32/shell32/CDefView.cpp b/dll/win32/shell32/CDefView.cpp index 33e5b5a760..32f7e153e7 100644 --- a/dll/win32/shell32/CDefView.cpp +++ b/dll/win32/shell32/CDefView.cpp @@ -418,7 +418,7 @@ HRESULT CDefView::IncludeObject(PCUITEMID_CHILD pidl) { TRACE("ICommDlgBrowser::IncludeObject pidl=%p\n", pidl); ret = m_pCommDlgBrowser->IncludeObject(this, pidl); - TRACE("--0x%08x\n", ret); + TRACE("-- returns 0x%08x\n", ret); }
return ret; @@ -432,7 +432,7 @@ HRESULT CDefView::OnDefaultCommand() { TRACE("ICommDlgBrowser::OnDefaultCommand\n"); ret = m_pCommDlgBrowser->OnDefaultCommand(this); - TRACE("-- returns %08x\n", ret); + TRACE("-- returns 0x%08x\n", ret); }
return ret; @@ -2082,7 +2082,7 @@ HRESULT WINAPI CDefView::TranslateAccelerator(LPMSG lpmsg) if (::TranslateAcceleratorW(m_hWnd, m_hAccel, lpmsg) != 0) return S_OK;
- TRACE("-- key=0x04%lx\n", lpmsg->wParam) ; + TRACE("-- key=0x%04lx\n", lpmsg->wParam) ; }
return m_pShellBrowser->TranslateAcceleratorSB(lpmsg, 0); @@ -3179,7 +3179,7 @@ HRESULT WINAPI CDefView::Unfreeze(DWORD dwFreeze)
HRESULT WINAPI CDefView::SetAdvise(DWORD aspects, DWORD advf, IAdviseSink *pAdvSink) { - FIXME("partial stub: %p %08x %08x %p\n", this, aspects, advf, pAdvSink); + FIXME("partial stub: %p 0x%08x 0x%08x %p\n", this, aspects, advf, pAdvSink);
/* FIXME: we set the AdviseSink, but never use it to send any advice */ m_pAdvSink = pAdvSink; diff --git a/ntoskrnl/wmi/smbios.c b/ntoskrnl/wmi/smbios.c index 8f353fd5b3..389678d4c0 100644 --- a/ntoskrnl/wmi/smbios.c +++ b/ntoskrnl/wmi/smbios.c @@ -229,7 +229,7 @@ WmipQueryRawSMBiosTables( Status = WmipGetRawSMBiosTableData(OutBuffer ? &TableData : NULL, &TableSize); if (!NT_SUCCESS(Status)) { - DPRINT1("WmipGetRawSMBiosTableData failed: 0x08lx\n", Status); + DPRINT1("WmipGetRawSMBiosTableData failed: 0x%08lx\n", Status); return Status; }
diff --git a/win32ss/user/ntuser/cursoricon.c b/win32ss/user/ntuser/cursoricon.c index a6dd22074e..80a15a39b0 100644 --- a/win32ss/user/ntuser/cursoricon.c +++ b/win32ss/user/ntuser/cursoricon.c @@ -482,7 +482,7 @@ NtUserGetIconInfo(
if (!NT_SUCCESS(Status)) { - WARN("Status: 0x%08x.\n", Status); + WARN("Status: 0x%08lx\n", Status); SetLastNtError(Status); goto leave; } @@ -1334,7 +1334,7 @@ IntSetAconData( /* Check if the flags are valid */ if (pcdFrame->CURSORF_flags & ~(CURSORF_USER_MASK|CURSORF_ACONFRAME)) { - ERR("Invalid flags for acon frame %u: 0x%lx\n", + ERR("Invalid flags for acon frame %u: 0x%08lx\n", i, pcdFrame->CURSORF_flags); goto Cleanup; } @@ -1436,7 +1436,7 @@ UserSetCursorIconData( &atomModName); if (!NT_SUCCESS(status)) { - ERR("Failed to create atom from module name '%wZ': %0x8lx\n", + ERR("Failed to create atom from module name '%wZ': 0x%08lx\n", pustrModule, status); return FALSE; } @@ -2179,7 +2179,7 @@ NtUserGetCursorFrameInfo(
if (!NT_SUCCESS(Status)) { - WARN("Status: 0x%08x.\n", Status); + WARN("Status: 0x%08lx\n", Status); SetLastNtError(Status); ret = NULL; }
-
Serge Gautherie