On Mon, 26 Dec 2011 16:21:57 +0100 Guillaume Friloux <[email protected]> wrote:
> On 26/12/2011 16:05, Guillaume Friloux wrote: > > Hello e people. > > > > If evas cant find the type of a file, it will try to use > > evas_image_loader, but not everyone has it. > > This will not block anything or crash, because the error is handled, > > but it is somehow ugly because you spend time building the command > > line and gets ugly messages to stderr. > > > > So here is a very simple patch to save some cycles in case you dont > > have the binary, and less spams your terminal. > > > > Please review it, it is my very first patch for evas and i know > > nothing about evas (i reported the bug and been asked to do the patch) > > > > We should save a lot more time by checking the existence of > > evas_image_loader once at startup and keep that information in memory > > instead of doing the check everytime, so i think some people here can > > do better than me with that > Please, nobody touch this patch, it only work if you dont have > evas_image_loader. If you do have it, it will make _load() for > evas_image_load_generic.c to never work! > > i need punishment. oh so I should commit it? COMMITTING ------------------------------------------------------------------------------ Write once. Port to many. Get the SDK and tools to simplify cross-platform app development. Create new or port existing apps to sell to consumers worldwide. Explore the Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join http://p.sf.net/sfu/intel-appdev _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
