[EMAIL PROTECTED] wrote: >> Enlightenment CVS committal >> >> Author : dj2 >> Project : e17 >> Module : libs/efreet >> >> Dir : e17/libs/efreet/src/lib >> >> >> Modified Files: >> efreet_ini.c >> >> >> Log Message: >> - if we can't parse the INI file we can't set ini->data so return NULL >> instead of an invalid ini file > > Are you sure this has no consequences? I'm pretty sure I did it that way > for a reason (maybe parse empty or non-existing file?) >
We'll have to wait to see if anyone complains. The problem is, we can't tell if issue is failed parsing or non-existing file. We may need to put a check in there for non-existant file. But, I don't think we should be pretending everything is good if the file fails to parse. dan ------------------------------------------------------------------------- This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel