On Thu, Sep 6, 2012 at 8:54 AM, Carsten Haitzler <ras...@rasterman.com> wrote: > On Thu, 6 Sep 2012 01:14:34 +0900 Daniel Juyung Seo <seojuyu...@gmail.com> > said: > >> I am sick and tired of hearing that "EFL code sucks" because of those >> missing exception handlings. > > and do these people bitch about libc too? checking the programmers bad and > non-sensical input is a nicety we do in some places. it is a NICETY! not a > requirement. libc sees it as a luxury and almost never does it. my bet is they > don't complain about libc. they simply want to pass blame away from their own > mistake and thus will say "well efl sucks".
libc is so low level. That's the difference between libc and efl. > > if we do any checking it should be consistent throughout a library. e_dbus > doesn't check most input as it's a thin wrapper. any checks should use > standard/common infrastructure where appropriate so it can be a macro and > compiled out for speed reasons or change behavior (eg call abort() on every > incorrect input to guarantee you get a crash and a backtrace to fix it). a > simple 'if (!string) return NULL;' is more likely to hide nasty bugs and make > them much harder to find. there is no complaint to stout/err. there is no way > to force a backtrace (abort) etc. This is exactly what I agree. Be consistent. So what's your opinion? Check NULL in EFL or not? Daniel Juyung Seo (SeoZ) > > -- > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > The Rasterman (Carsten Haitzler) ras...@rasterman.com > ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
