Merge pull request #5358 from WalterBright/fix15578.d

fix Issue 15578 - Should have access to namespace private symbols fro…
dlang · Jan 23, 2016 · 22b1c0b · 22b1c0b
2 parents c6ac10a + 92e637e
commit 22b1c0b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/dsymbol.d b/src/dsymbol.d
@@ -1315,7 +1315,8 @@ public:
                         a = mergeOverloadSet(ident, a, s);
                     s = a;
                 }
-                if (!(flags & IgnoreErrors) && s.prot().kind == PROTprivate && !s.parent.isTemplateMixin())
+                if (!(flags & IgnoreErrors) && s.prot().kind == PROTprivate &&
+                    !s.parent.isTemplateMixin() && !s.parent.isNspace())
                 {
                     if (!s.isImport())
                         error(loc, "%s %s is private", s.kind(), s.toPrettyChars());

diff --git a/test/compilable/test15578.d b/test/compilable/test15578.d
@@ -0,0 +1,10 @@
+private:
+    int j;
+    extern(C++, ns) int k;
+
+void f()
+{
+    j = 0; // works as expected
+    k = 0; // Error: variable foo.ns.k is private
+}
+