Alexandre Julliard wrote:
Mike McCormack <[EMAIL PROTECTED]> writes:
I thought that crashing on out of memory conditions was a valid method
of error handling in the Julliard handbook of coding, so I removed a
pointless check :)
If we're going to mandate proper checking of memory allocation, then
we need to make a janitorial task to do so, as there's many, many
places in the code it's not done.
Well yes, ultimately we will need to have proper checks everywhere.
It's not really high priority, and I don't think there's a need to
make an effort in this direction at this point. But at least we can
Well, it should be pretty easy to write a short smatch script to find
that occurences. Afair the smatch guys wrote one for the Linux kernel
which would need only small adaptations. And coincidently i have this
weekend a long flight trip on my schedule ... . And we seem to have some
people now doing janitorial work. I figure 2-3 people would maybe jump
on it, maybe even new people as the task isn't hard at all. If we get
new people it's nice cause they get to see a lot of wine code and get
used to the patch acceptance policy ;). Maybe they get motivated to try
there luck on other wine tasks too.
avoid removing the checks that are already in place...
bye
michael
--
Michael Stefaniuc Tel.: +49-711-96437-199
System Administration Fax.: +49-711-96437-111
Red Hat GmbH Email: [EMAIL PROTECTED]
Hauptstaetterstr. 58 http://www.redhat.de/
D-70178 Stuttgart