X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/e4a81a2e96b5a349ee9430b91d36e7e8a21d7976..2e1a688e3e9c6c265620e4f05d579077367e9fdb:/src/motif/colour.cpp?ds=inline diff --git a/src/motif/colour.cpp b/src/motif/colour.cpp index 9963f79b6b..cf9f93c796 100644 --- a/src/motif/colour.cpp +++ b/src/motif/colour.cpp @@ -57,12 +57,12 @@ wxColour::wxColour (const wxColour& col) wxColour& wxColour::operator =(const wxColour& col) { - m_red = col.m_red; - m_green = col.m_green; - m_blue = col.m_blue; - m_isInit = col.m_isInit; - m_pixel = col.m_pixel; - return *this; + m_red = col.m_red; + m_green = col.m_green; + m_blue = col.m_blue; + m_isInit = col.m_isInit; + m_pixel = col.m_pixel; + return *this; } void wxColour::InitFromName(const wxString& col) @@ -109,8 +109,8 @@ void wxColour::Set (unsigned char r, unsigned char g, unsigned char b) int wxColour::AllocColour(WXDisplay* display, bool realloc) { if ((m_pixel != -1) && !realloc) - return m_pixel; - + return m_pixel; + XColor color; color.red = (unsigned short) Red (); color.red |= color.red << 8; @@ -118,20 +118,20 @@ int wxColour::AllocColour(WXDisplay* display, bool realloc) color.green |= color.green << 8; color.blue = (unsigned short) Blue (); color.blue |= color.blue << 8; - + color.flags = DoRed | DoGreen | DoBlue; - + WXColormap cmap = wxTheApp->GetMainColormap(display); - + if (!XAllocColor ((Display*) display, (Colormap) cmap, &color)) { - m_pixel = wxGetBestMatchingPixel((Display*) display, &color,(Colormap) cmap); - return m_pixel; + m_pixel = wxGetBestMatchingPixel((Display*) display, &color,(Colormap) cmap); + return m_pixel; } else { - m_pixel = (int) color.pixel; - return m_pixel; + m_pixel = (int) color.pixel; + return m_pixel; } } @@ -146,30 +146,30 @@ But in many cases, that is still better than always using black. Chris Breeze Improvements: 1) More efficient calculation of RGB distance of colour cell from - the desired colour. There is no need to take the sqrt of 'dist', and - since we are only interested in the top 8-bits of R, G and B we - can perform integer arithmetic. +the desired colour. There is no need to take the sqrt of 'dist', and +since we are only interested in the top 8-bits of R, G and B we +can perform integer arithmetic. 2) Attempt to allocate a read-only colour when a close match is found. - A read-only colour will not change. +A read-only colour will not change. 3) Fall back to the closest match if no read-only colours are available. -Possible further improvements: -1) Scan the lookup table and sort the colour cells in order of -increasing - distance from the desired colour. Then attempt to allocate a -read-only - colour starting from the nearest match. -2) Linear RGB distance is not a particularly good method of colour -matching - (though it is quick). Converting the colour to HLS and then comparing - may give better matching. + Possible further improvements: + 1) Scan the lookup table and sort the colour cells in order of + increasing + distance from the desired colour. Then attempt to allocate a + read-only + colour starting from the nearest match. + 2) Linear RGB distance is not a particularly good method of colour + matching + (though it is quick). Converting the colour to HLS and then comparing + may give better matching. -------------------------------------------*/ int wxGetBestMatchingPixel(Display *display, XColor *desiredColor, Colormap cmap) { if (cmap == (Colormap) NULL) - cmap = (Colormap) wxTheApp->GetMainColormap(display); - + cmap = (Colormap) wxTheApp->GetMainColormap(display); + int numPixVals = XDisplayCells(display, DefaultScreen (display)); int mindist = 256 * 256 * 3; int bestpixel = (int) BlackPixel (display, DefaultScreen (display)); @@ -177,21 +177,21 @@ int wxGetBestMatchingPixel(Display *display, XColor *desiredColor, Colormap cmap int green = desiredColor->green >> 8; int blue = desiredColor->blue >> 8; const int threshold = 2 * 2 * 3; // allow an error of up to 2 in R,G & B - + for (int pixelcount = 0; pixelcount < numPixVals; pixelcount++) { XColor matching_color; matching_color.pixel = pixelcount; XQueryColor(display,cmap,&matching_color); - + int delta_red = red - (matching_color.red >> 8); int delta_green = green - (matching_color.green >> 8); int delta_blue = blue - (matching_color.blue >> 8); - + int dist = delta_red * delta_red + - delta_green * delta_green + - delta_blue * delta_blue; - + delta_green * delta_green + + delta_blue * delta_blue; + if (dist <= threshold) { // try to allocate a read-only colour...