> Sven Neumann wrote:
> > 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...
> 
> Removing all code for now should be fine.
> 
> --
> Best regards,
>    Denis Oliver Kropp

No problem, I'll remove all of the JPEG related code.
Thanks
Phil
_______________________________________________
directfb-dev mailing list
directfb-dev@directfb.org
http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev

Reply via email to