Re: [PATCH v5 0/7] MR2221: imm32/tests: Test ImmInstallIMEW with an actual IME.

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about tools/makedep.c:
> */
>  static void output_source_spec( struct makefile *make, struct incl_file *source, const char *obj )
>  {
> - const char *dll_name, *obj_name, *res_name, *output_file, *output_rsrc, *debug_file;
> struct strarray imports = get_expanded_file_local_var( make, obj, "IMPORTS" );
> struct strarray dll_flags = empty_strarray;
> struct strarray default_imports = empty_strarray;
> struct strarray all_libs, dep_libs;
> - const char *dll_name, *obj_name, *res_name, *output_file, *debug_file;
> unsigned int arch;

Please avoid moving entire lines when changing something, it makes it harder to see what actually changed.
Sorting variable declarations by line length is a silly waste of time IMNSHO.