Do not return disconnected IdDict from Docs.meta (JuliaLang#38821)

Fixes JuliaLang#38819 Fixes timholy/Revise.jl#583
ElOceanografo · May 4, 2021 · 297597d · 297597d
1 parent f3f4594
commit 297597d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/base/docs/Docs.jl b/base/docs/Docs.jl
@@ -73,7 +73,12 @@ const modules = Module[]
 const META = gensym(:meta)
 const METAType = IdDict{Any,Any}
 
-meta(m::Module) = isdefined(m, META) ? getfield(m, META)::METAType : METAType()
+function meta(m::Module)
+ if !isdefined(m, META)
+ initmeta(m)
+ end
+ return getfield(m, META)::METAType
+end
 
 function initmeta(m::Module)
  if !isdefined(m, META)

diff --git a/test/docs.jl b/test/docs.jl
@@ -73,6 +73,11 @@ end
 @test docstrings_equal(@doc(ModuleMacroDoc), doc"I am a module")
 @test docstrings_equal(@doc(ModuleMacroDoc.@m), doc"I am a macro")
 
+# issue #38819
+
+module NoDocStrings end
+@test meta(NoDocStrings) === getfield(NoDocStrings, Base.Docs.META)
+
 # General tests for docstrings.
 
 const LINE_NUMBER = @__LINE__() + 1