Comment 1 for bug 1274516

Revision history for this message
Peter TB Brett (peter-b) wrote :

Hi Vladimir,

Why did you decide to fix the crash in this way? Neither the commit message nor the patch itself is very illuminating.

I actually disagree with your justification in the patch that "Decision of what to do with any files is not in the competence of
edarenderer." Working out what to do if an image can't be loaded and/or displayed isn't really in the competence of libgeda *or* libgedacairo - ideally, it should be decided by the application that's using the renderer (for example, it might want to supply a different fallback image depending on the current icon theme it's using).

I suppose I'd better file another bug about this.

Thanks,

Peter