Brett Nash wrote: > Following on the heels of my ever so successful set of evas patches, > here are a set for e. Some of these fix real bugs, most are obviously > masked by irregular occurrence (esp the uninitialised pointer ones). > > I've left them all in one glob this time. > > Anyway changes are: > e_actions.c: [BUG] Get rid of typo: > if (ret>1); -> if (ret>1) > > e_wizard.c: [BUG] evas_key_modifier_is_set takes a Evas_Modifier*, not > an Evas* for first parameter. Note evas_key_modifier_is_set does not > check that the first parameter is an Evas_Modifier, so will happily use > the Evas* as and Evas_Modifier*. Hilarity ensures. > > e_fm_custom.c: [Avoid GCCism] Don't do maths on void*s, cast (char*) > first. > e_int_config_theme.c: As above; void*->char* > > e_int_border_prop.c: [64bit] Just check data2 directly, don't assign to > int first. > > e_spectrum.c: Init vz to 0 always. > > e_int_config_mousebindings.c: Set ev, bw, action and params to NULL > before using. > > multiple modules [BUG]: > Set zone to NULL. Used uninitialised multiple places > > multiple places [C89]: > More C89 compliance with function declarations (make them a C89 > prototype). > > Regards, > nash > Brett,
Most of these went into cvs. Some of the patches were slightly modified, but in general, Nice Work :) devilhorns ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
