From: Eric Karl <ericrk@chromium.org>
Date: Mon, 17 Apr 2017 19:04:16 +0000 (-0700)
Subject: Pass LC_ALL_MASK to newlocale
X-Git-Tag: accepted/tizen/5.0/unified/20181102.025319~36^2~748
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0d2c9a252e0f113a70842280d0ef7190a062fa89;p=platform%2Fupstream%2FlibSkiaSharp.git

Pass LC_ALL_MASK to newlocale

This change fixes an issue with GrAutoLocaleSetter, which was
using the C library LC_ALL mask with newlocale. Instead the
xlocale LC_ALL_MASK should be used (these do not match on all
platforms).

See: http://man7.org/linux/man-pages/man3/newlocale.3.html

Bug: 711683
Change-Id: I6dba140e72b095f75667f0dcc98480d8d4562191
Reviewed-on: https://skia-review.googlesource.com/13601
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
---

diff --git a/src/gpu/GrAutoLocaleSetter.h b/src/gpu/GrAutoLocaleSetter.h
index cec041e..9cfa637 100644
--- a/src/gpu/GrAutoLocaleSetter.h
+++ b/src/gpu/GrAutoLocaleSetter.h
@@ -54,7 +54,7 @@ public:
             name = nullptr;
         }
 #endif
-        fLocale = newlocale(LC_ALL, name, 0);
+        fLocale = newlocale(LC_ALL_MASK, name, 0);
         if (fLocale) {
             fOldLocale = uselocale(fLocale);
         } else {