gnash-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnash-commit] gnash ChangeLog backend/render_handler_agg.cpp ...


From: Sandro Santilli
Subject: Re: [Gnash-commit] gnash ChangeLog backend/render_handler_agg.cpp ...
Date: Wed, 9 May 2007 12:14:40 +0200

On Wed, May 09, 2007 at 12:04:27PM +0200, Udo Giacomozzi wrote:
> Hello Sandro,
> 
> Wednesday, May 9, 2007, 11:54:44 AM, you wrote:
> SS>                 * backend/render_handler_agg.{cpp,h} 
> (create_render_handler_agg):
> SS>                   Take pixelformat by *const* pointer, do NOT abort when 
> the given
> SS>                   pixelformat is unsupported, just return NULL.
> 
> Have you checked that GTK and Gnash core can deal with that situation
> (that the render handler is NULL)?
> 
> I see set_render_handler(NULL) is called (didn't look further).

No, I haven't checked. Yes, an abort() is better then a segfault, I'll take
a look...

--strk;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]