<div>On Fri, Mar 3, 2017 at 2:31 PM Alexandre Julliard <<a href="mailto:julliard@winehq.org">julliard@winehq.org</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Chris Morgan <<a href="mailto:chmorgan@gmail.com" class="gmail_msg" target="_blank">chmorgan@gmail.com</a>> writes:<br class="gmail_msg">
<br class="gmail_msg">
> Avoids the ambiguity with non-alphanumeric hex values.<br class="gmail_msg">
><br class="gmail_msg">
> Signed-off-by: Chris Morgan <<a href="mailto:chmorgan@gmail.com" class="gmail_msg" target="_blank">chmorgan@gmail.com</a>><br class="gmail_msg">
> ---<br class="gmail_msg">
>  dlls/ntdll/file.c | 2 +-<br class="gmail_msg">
>  1 file changed, 1 insertion(+), 1 deletion(-)<br class="gmail_msg">
><br class="gmail_msg">
> diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c<br class="gmail_msg">
> index fd7f3dd..4d081b7 100644<br class="gmail_msg">
> --- a/dlls/ntdll/file.c<br class="gmail_msg">
> +++ b/dlls/ntdll/file.c<br class="gmail_msg">
> @@ -1567,7 +1567,7 @@ static NTSTATUS server_ioctl_file( HANDLE handle, HANDLE event,<br class="gmail_msg">
>      SERVER_END_REQ;<br class="gmail_msg">
><br class="gmail_msg">
>      if (status == STATUS_NOT_SUPPORTED)<br class="gmail_msg">
> -        FIXME("Unsupported ioctl %x (device=%x access=%x func=%x method=%x)\n",<br class="gmail_msg">
> +        FIXME("Unsupported ioctl 0x%x (device=0x%x access=0x%x func=0x%x method=0x%x)\n",<br class="gmail_msg">
<br class="gmail_msg">
That's mostly a matter of taste, so I don't think it's necessary to<br class="gmail_msg">
change it.<br class="gmail_msg">
<br class="gmail_msg">
--<br class="gmail_msg">
Alexandre Julliard<br class="gmail_msg">
<a href="mailto:julliard@winehq.org" class="gmail_msg" target="_blank">julliard@winehq.org</a></blockquote><div><br></div><div>Ok. I think most of the rest of the file uses 0x and it cost me an hour or so before I figured out they weren't decimal values as they didn't include any letters. And the list of FSCTL options in headers uses decimal so that was another layer of confusion.</div><div><br></div><div>There is one more case that doesn't use 0x that I found after submitting the patch.</div><div><br></div><div>I'd still like to advocate for being explicit because it isn't always obvious what base is used.</div><div><br></div><div>Chris</div><div><br></div></div></div>