isdeprecated does not resolve binding

JuliaLang · Nov 8, 2018 · b662ed4 · b662ed4
1 parent 5922b5e
commit b662ed4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/module.c b/src/module.c
@@ -534,8 +534,12 @@ JL_DLLEXPORT void jl_deprecate_binding(jl_module_t *m, jl_sym_t *var, int flag)
 
 JL_DLLEXPORT int jl_is_binding_deprecated(jl_module_t *m, jl_sym_t *var)
 {
-    jl_binding_t *b = jl_get_binding(m, var);
-    return b && b->deprecated;
+    if (jl_binding_resolved_p(m, var))
+    {
+        jl_binding_t *b = jl_get_binding(m, var);
+        return b && b->deprecated;
+    }
+    return 0;
 }
 
 extern const char *jl_filename;

diff --git a/test/reflection.jl b/test/reflection.jl
@@ -809,3 +809,6 @@ f20872(::Val, ::Val) = false
 @test which(f20872, Tuple{Val,Val}).sig == Tuple{typeof(f20872), Val, Val}
 @test which(f20872, Tuple{Val,Val{N}} where N).sig == Tuple{typeof(f20872), Val, Val}
 @test_throws ErrorException which(f20872, Tuple{Any,Val{N}} where N)
+
+# make sure checking if a binding is deprecated does not resolve it
+@test !(!Base.isdeprecated(Main, :sin) && !Base.isbindingresolved(Main, :sin))