On 08.03.2013 22:13, Ian MacArthur wrote: > > On 8 Mar 2013, at 17:31, Greg Ercolano wrote: > >> A request to vote on the addition of the gleam and grad patches to FLTK. >> >> There's two STRs for this: >> http://www.fltk.org/str.php?L2672 >> http://www.fltk.org/str.php?L2675 >> >> Would like to combine and apply. Code seems OK, looking for consensus >> and comments on patches. > > I've not looked at the patches in detail recently - looked at the gleam patch > a while back, but Edmanuel has been working on it more since then... > > If it looks OK to others, then I'd be +1 on adding it, as more schemes would > probably be a popular option!
Same here, +1 on adding both new schemes, if they are okay for inclusion. I posted a bug report and a few comments to STR #2672. I did NOT test the grad patch yet (STR #2675). >> My only comment is I'd like to modify the unittests program to have a test >> for all our scheme options (plastic, gtk+, etc) so it can be used to look for >> artifacts. Basically taking STR#2672's test/gleam application and moving it >> into unittests as one of the test screens (with some mods). > > This sounds like a good idea in any case: this is what the unittests > framework was meant for really, I think! > (I mean, adding on all sorts of useful testing features without having to ad > new test programs directly.) +1 (and since this is done already, thanks for doing it, Greg) Albrecht _______________________________________________ fltk-dev mailing list fltk-dev@easysw.com http://lists.easysw.com/mailman/listinfo/fltk-dev