Just the suggestion that as all standards support using %#08x to prefix
with 0x (prefix output capitalization follows format letter
capitalization) and would be preferable to hacking the text 0x onto the
format %08X, doing all of the formatting work with the format flags.
My awareness and attitude to modifying output presentation using only
formats was hardened by those not using date formats to modify date
presentation during projects prior to Y2K!
[I want to scream and rant when I see imbeciles still producing output
using meaningless 10/11/12 date formats, on systems and especially on
web sites, where JavaScript supports perfectly nice internationalized
formatting that shows dates and times in my zone and preferred formats!]
On 2022-01-15 12:04, Lavrentiev, Anton (NIH/NLM/NCBI) [C] wrote:
So? With %X (capital X) the alternate form has the prefix 0X capital, too; and
it's really hard to read.
IDK what is exactly your point that you are trying to make, is my patch somehow
incorrect, or what?
Anton Lavrentiev
Contractor NIH/NLM/NCBI
-----Original Message-----
From: Brian Inglis <brian.ing...@systematicsw.ab.ca>
Sent: Friday, January 14, 2022 11:38 PM
To: cygwin-patches@cygwin.com
Subject: [EXTERNAL] Re: [PATCH 2/7] Use matching format for NTSTATUS
CAUTION: This email originated from outside of the organization. Do not click
links or
open attachments unless you recognize the sender and are confident the content
is safe.
See fprintf(3p) POSIX:
# Specifies that the value is to be converted to an alternative form.
...
For x or X conversion specifiers, a non-zero result shall have 0x
(or 0X) prefixed to it.
On 2022-01-14 15:10, Anton Lavrentiev via Cygwin-patches wrote:
---
winsup/cygwin/libc/minires-os-if.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/winsup/cygwin/libc/minires-os-if.c
b/winsup/cygwin/libc/minires-os-if.c
index 666a008de..6e17de0b8 100644
--- a/winsup/cygwin/libc/minires-os-if.c
+++ b/winsup/cygwin/libc/minires-os-if.c
@@ -359,7 +359,7 @@ static void get_registry_dns(res_state statp)
status = RtlCheckRegistryKey (RTL_REGISTRY_SERVICES, keyName);
if (!NT_SUCCESS (status))
{
- DPRINTF (statp->options & RES_DEBUG, "RtlCheckRegistryKey: status %p\n",
+ DPRINTF (statp->options & RES_DEBUG, "RtlCheckRegistryKey: status
0x%08X\n",
DPRINTF (statp->options & RES_DEBUG, "RtlCheckRegistryKey:
status %#08x\n",
status);
return;
}
@@ -381,7 +381,7 @@ static void get_registry_dns(res_state statp)
if (!NT_SUCCESS (status))
{
DPRINTF (statp->options & RES_DEBUG,
- "RtlQueryRegistryValues: status %p\n", status);
+ "RtlQueryRegistryValues: status 0x%08x\n", status);
"RtlQueryRegistryValues: status %#08x\n", status);
return;
}
--
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada
This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.
[Data in binary units and prefixes, physical quantities in SI.]
