[PATCH] Cygwin: path: Convert type of variable 'remlen' to DWORD.

Corinna Vinschen corinna-cygwin@cygwin.com
Mon Mar 14 18:00:19 GMT 2022


On Mar 14 20:36, Takashi Yano wrote:
> - Variable remlen stores the return value of QueryDosDeviceW(), so
>   it is better to be DWORD.
> ---
>  winsup/cygwin/path.cc | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/winsup/cygwin/path.cc b/winsup/cygwin/path.cc
> index eceafbbcf..e370843ee 100644
> --- a/winsup/cygwin/path.cc
> +++ b/winsup/cygwin/path.cc
> @@ -3523,8 +3523,7 @@ restart:
>  			{(WCHAR) towupper (upath.Buffer[4]), L':', L'\0'};
>  		      WCHAR remote[MAX_PATH];
>  
> -
> -		      int remlen = QueryDosDeviceW (drive, remote, MAX_PATH);
> +		      DWORD remlen = QueryDosDeviceW (drive, remote, MAX_PATH);
>  		      if (remlen < 3)
>  			goto file_not_symlink; /* fallback */
>  		      remlen -= 2; /* Two L'\0' */
> -- 
> 2.35.1

Thanks, please push.


Corinna


More information about the Cygwin-patches mailing list