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

.------------------------------------------.
| DirectFB - Hardware accelerated graphics |
| http://www.directfb.org/                 |
"------------------------------------------"
_______________________________________________
directfb-dev mailing list
directfb-dev@directfb.org
http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev

Reply via email to