Wine patch code review or what to do if patches get stuck?

Paul Vriens paul.vriens.wine at gmail.com
Tue Apr 21 10:01:42 CDT 2009


Reece Dunn wrote:
> 2009/4/21 Jeremy White <jwhite at winehq.org>:
>>> Sometimes "no feedback" means that you just dont know - is there
>>> something wrong? Or maybe your patch wos not understood correctly? Or
>>> there are doubts that what you did is right? Or what? It would be
>>> better to have feedback like "are you serious?" than nothing at all :)
>> I've tried to update the 'why was my patch ignored' section in the Wiki:
>>  http://wiki.winehq.org/DeveloperFaq#head-1e5a50563dbdb221702cbde246b23f5de08a75e9
>> to reflect what I understand from Alexandre.
>>
>> Not sure if any of it applies in your case, but I thought it'd be good
>> to have it in the mailing list archives.
> 
> I found http://wiki.jswindle.com/index.php/Coding_Hints:Part_1 that
> may be useful as a checklist.
> 
> I was trying to find the interview with Alexandre where he lists 10
> things that cause a patch to be rejected (e.g. using C++ comments).
> Does anyone know the link for this?
> 
> - Reece
> 
> 
Do you mean:

http://winehq.org/wwn/353#News:%20WineConf%202008

(It's not 10 reasons though).

-- 
Cheers,

Paul.



More information about the wine-devel mailing list