[Bf-committers] Re: [Bf-blender-cvs] CVS commit: blender/source/blender/blenloader/intern writefile.c

Alexander Ewering blender at instinctive.de
Fri Oct 27 22:14:05 CEST 2006


On Fri, 27 Oct 2006, Martin Poirier wrote:

> I disagree, while simply bailing out on NULL might not
> be the optimal solution, it's much better then
> crashing.

Well, for example, in the writefile.c case (last commit), it would probably
mean that a file full of garbage is written, and Blender does not tell the
user (as it just quits writing the file with all the returns).

Maybe in such a case, at least some thought should be put into a meaningful
error message. Though, with so little memory, the error() dialog would
probably not work, either, and well, the kernel would probably be killing
blender anyway ;)

| alexander ewering              instinctive mediaworks
| ae[@]instinctive[.]de   http://www[.]instinctive[.]de


More information about the Bf-committers mailing list