New uninstaller

Jonathan Ernst Jonathan at ErnstFamily.ch
Tue Mar 8 03:18:50 CST 2005


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 !
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Ceci est une partie de message
	=?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20050308/380b7fad/attachment.pgp


More information about the wine-devel mailing list