On Wed, 16 Feb 2011 13:31:35 +0900 Jeonghyun Yun <[email protected]> said:
thanks! in svn it is! :) > Hello. > > I agree to your comment. > I implemented the same logic change both rle 4bit and rle 8bit. > But I can't find another path any more. > > Please check this patch again. > > Thanks. > > -----Original Message----- > From: Carsten Haitzler (The Rasterman) [mailto:[email protected]] > Sent: Tuesday, February 15, 2011 8:35 PM > To: Jeonghyun Yun > Cc: [email protected] > Subject: Re: [E-devel] [Patch] evas_image_load_bmp small patch > > On Thu, 10 Feb 2011 12:00:13 +0900 Jeonghyun Yun <[email protected]> > said: > > > Hello. > > > > When I tested some bmp files on evas, I found one bmp file is not > displayed. > > But this bmp file is displayed on windows and linux gimp and etc. > > So I checked this bmp file, this file have 208 data per line despite 207 > > pixel per line!! > > > > I fixed code to skip the data when data is more than width instead of > break. > > I think this is not bug but evas policy problem. > > > > I attached patch and bmp file. > > Can anybody check this issue? > > what kind of nutty app produced such a bmp? thats a pretty broken bmp... but > i > guess there is no harm in making it more robust to handle such unsociable > bmp > files... but still.. thats one rude bmp! :) > > now about the patch - you only change the rle 8bit/pixel path -w hat about > all > the other modes/paths? it's a bit poor to have 1 path work differently to > all > the other ones? could you also implement the same logic change in the other > decode paths? (rle 4bit is what i'm looking at specifically as it has the > same > logic/issue) > > -- > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > The Rasterman (Carsten Haitzler) [email protected] -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) [email protected] ------------------------------------------------------------------------------ The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE: Pinpoint memory and threading errors before they happen. Find and fix more than 250 security defects in the development cycle. Locate bottlenecks in serial and parallel code that limit performance. http://p.sf.net/sfu/intel-dev2devfeb _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
