Re: sRGB color support in NS port [PATCH]

[Steve Purcell]

On 26 Dec 2013, at 12:12, Steve Purcell <address@hidden> wrote:

> Thanks to Jan’s work, it's just a matter of setting ns-use-srgb-colorspace to 
> YES instead of NO in src/nsterm.m. Corresponding patch is below:



Oops, missed a docstring. Here’s a better patch:


fb80128e4ab8d1884a25b89a9ee5b166cd8595a5 HEAD master
Author: Steve Purcell <address@hidden>
Date:   Thu Dec 26 12:08:20 2013 +0000

    Enable ns-use-srgb-colorspace by default
    
    See thread 
https://lists.gnu.org/archive/html/emacs-devel/2013-12/msg00706.html

2 files changed, 3 insertions(+), 3 deletions(-)
 etc/NEWS     | 2 +-
 src/nsterm.m | 4 ++--

        Modified   etc/NEWS
diff --git a/etc/NEWS b/etc/NEWS
index 5aedab6..03ea2e4 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -1248,7 +1248,7 @@ Customize `ns-use-native-fullscreen' to change style.  
For >= 10.7
 native is the default.
 
 ** OSX >= 10.7 can use sRGB colorspace.
-Customize `ns-use-srgb-colorspace' to change style.  nil is the default.
+Customize `ns-use-srgb-colorspace' to change style.  t is the default.
 Note: This does not apply to images.
 
 
        Modified   src/nsterm.m
diff --git a/src/nsterm.m b/src/nsterm.m
index f2aef42..635f579 100644
--- a/src/nsterm.m
+++ b/src/nsterm.m
@@ -7647,8 +7647,8 @@ Default is t for OSX >= 10.7, nil otherwise. */);
   DEFVAR_BOOL ("ns-use-srgb-colorspace", ns_use_srgb_colorspace,
      doc: /*Non-nil means to use sRGB colorspace on OSX >= 10.7.
 Note that this does not apply to images.            
-This variable is ignored on OSX < 10.7 and GNUStep.  Default is nil. */);
-  ns_use_srgb_colorspace = NO;
+This variable is ignored on OSX < 10.7 and GNUStep.  Default is t. */);
+  ns_use_srgb_colorspace = YES;
 
   /* TODO: move to common code */
   DEFVAR_LISP ("x-toolkit-scroll-bars", Vx_toolkit_scroll_bars,