[quake3] Re: r1253 - in trunk: . code/renderer

[Ludwig Nussel]

Tim Angus wrote:
> DONOTREPLY at icculus.org wrote:
> > Author: ludwig
> > Date: 2008-02-12 05:03:21 -0500 (Tue, 12 Feb 2008)
> > New Revision: 1253
> > 
> > Added:
> >    trunk/code/renderer/tr_image_png.c
> >    trunk/code/renderer/tr_image_png.h
> > Modified:
> >    trunk/Makefile
> >    trunk/code/renderer/tr_image.c
> > Log:
> > move png support to separate file
> 
> Why?

It's so much code for a single file format that I've decided to
split it off:

$ wc -l code/renderer/tr_image.c code/renderer/tr_image_png.c
  2757 code/renderer/tr_image.c
  2474 code/renderer/tr_image_png.c

It's also easier to test separately then which was my motivation in
the first place. I'm currently running fuzzing tests against the
image loaders.

cu
Ludwig

-- 
 (o_   Ludwig Nussel
 //\   
 V_/_  http://www.suse.de/
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nuernberg)