Code quality (was Re: comctl32: Fix invalid syntax)

[Stefan Dösinger]

Am Samstag, 9. Februar 2008 02:13:34 schrieb L. Rahyen:
> On Friday February 8 2008 04:41:28 Bang Jun-young wrote:
> > That's the main reason why Wine keeps crashing every time I give it a
> > try with my Windows apps.
> > ...
> > I see something fundamentally wrong with  development process.
>
> 	I think that current development process isn't a problem at all. In fact
> AJ is very good at what he is doing!
> 	As far as I understand your patch ("comctl32: Fix invalid syntax.") was
> rejected just because you forgot to add proper (descriptive) changelog
> entry. What is the real problem is the lack of testers (who report
> regressions and bugs) and developers. This is why WINE still has a lot of
> bugs, regressions are quite common thing to happen, etc. To fix this, more
> people should read wine-patches and test patches before they are committed,
> more people should write bug reports, more people should be involved in the
> development, and so on. Unfortunately, not all people have enough time for
> such tasks. However, WINE is pretty usable today.
I'd say we need more tests. Any patch that breaks a test from our testsuite 
when AJ runs them prior to committing is rejected