Le jeudi 03 mars 2005 Ã 21:51 +0900, Mike McCormack a Ãcrit :
> Hi Jonathan,
> 
> You'll have more success getting this applied if you send each step as a 
> seperate patch.  I suggest starting by sending a single patch that 
> converts the uninstaller to use a dialog instead of a window, adds a 
> dialog resource in English and does nothing else.

Ok, once everything works I'll try to split it up as much as I can.

> 
> Once that patch is accepted, move onto the other tasks you have listed 
> above.  One step at a time makes the patch easier to review, and reduces 
> the chance that a single line mistake will stop your entire patch from 
> going in.
> 
> Additionally, there's no need to change the name of main.c to 
> uninstaller.c, that will just make it harder to see what has changed 
> between versions.

I thought it would be more homogeneous as taskmanager, winecfg where
using <program_name>.c files and that most of this file was rewritten by
the way. But I renamed it back in my latest test patch.
 

Thanks for your comments !

Attachment: signature.asc
Description: Ceci est une partie de message =?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=

Reply via email to