On Thu, 2010-07-29 at 11:31 +0100, Phil Edworthy wrote: > > > The sh772x gfx driver no longer uses libjpeg, so this patch removes > > > the #includes for jpeg related code. > > > > As far as I can see your patch only comments out the unused includes. > > Please submit a new patch that actually removes the lines. Thanks. > > I can do so if you want, but the reason I commented them out is that the > rest of the jpeg related code was commented out in an earlier patch. I guess > jpeg support may be added back in at a later time.
I guess the other commented out code should then be removed as well. DirectFB uses a version control system. There is no need to keep dead code around just in case that someone might want to add it later. But I'll leave this up to the DirectFB maintainers to decide... Sven _______________________________________________ directfb-dev mailing list directfb-dev@directfb.org http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev