Jason Edmeades : xcopy: Add support for COPYCMD override and fix /-y.

Alexander.Farber at nokia.com Alexander.Farber at nokia.com
Fri Mar 30 06:25:03 CDT 2007


Hello,

wouldn't it be nicer to use sizeof(copyCmd) here?

if (GetEnvironmentVariable(COPYCMD, copyCmd, MAXSTRING)) {

There are many spots like this in wcmdmain.c 

Regards
Alex

> -----Original Message-----
> From: wine-cvs-bounces at winehq.org 
> [mailto:wine-cvs-bounces at winehq.org] On Behalf Of ext Alexandre
Julliard
....
>  
> +    /* Preinitialize flags based on COPYCMD */
> +    if (GetEnvironmentVariable(COPYCMD, copyCmd, MAXSTRING)) {
> +        if (wcsstr(copyCmd, PROMPTSTR1) != NULL ||
> +            wcsstr(copyCmd, PROMPTSTR2) != NULL) {
> +            flags |= OPT_NOPROMPT;
> +        }
> +    }
> +
> 



More information about the wine-devel mailing list