On Wednesday 15 Sep 2004 07:30, Guillaume Laurent wrote: > On Wednesday 15 September 2004 05:05, Matthias Neeracher wrote: > > - My patch replaces references to Qt::whatever with the hardcoded > > color values. This is not all that pretty, but the probability of > > Qt::red being changed to a different RGB value are pretty low :-) > > Silvan, I see you applied this patch : please revert it. There's a > better way to fix this without resorting to ugly magic numbers (no > matter how unlikely they are to change).
Unless it's a very trivial better way, I suggest we leave the numbers. This file (colours.cpp) is full of magic numbers already, in the shape of several dozen RGB and HSV colour specifications. Chris ------------------------------------------------------- This SF.Net email is sponsored by: thawte's Crypto Challenge Vl Crack the code and win a Sony DCRHC40 MiniDV Digital Handycam Camcorder. More prizes in the weekly Lunch Hour Challenge. Sign up NOW http://ad.doubleclick.net/clk;10740251;10262165;m _______________________________________________ Rosegarden-devel mailing list [EMAIL PROTECTED] - use the link below to unsubscribe https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
