minor refactors

Author: aviatesk

src/Atom.jl

@@ -22,12 +22,12 @@ function __init__()
 
       # HACK: overloading this allows us to open remote files
       InteractiveUtils.eval(quote
-        function InteractiveUtils.edit(path::AbstractString, line::Integer=0)
+        function InteractiveUtils.edit(path::AbstractString, line::Integer = 0)
           if endswith(path, ".jl")
-              f = Base.find_source_file(path)
-              f !== nothing && (path = f)
+            f = Base.find_source_file(path)
+            f !== nothing && (path = f)
           end
-          $(msg)("openFile", Base.abspath(path), line-1)
+          $(msg)("openFile", Base.abspath(path), line - 1)
         end
       end)
     end

src/goto.jl

@@ -120,7 +120,7 @@ const PathItemsMaps = Dict{String, Vector{ToplevelItem}}
 """
     Atom.SYMBOLSCACHE
 
-"module" (`String`) ⟶ "path" (`String`) ⟶ "symbols" (`Vector{ToplevelItem}`) map
+"module" (`String`) ⟶ "path" (`String`) ⟶ "symbols" (`Vector{ToplevelItem}`) map.
 
 !!! note
     "module" should be canonical, i.e.: should be identical to names that are
@@ -146,26 +146,24 @@ function toplevelgotoitems(word, mod, path, text)
   return ret
 end
 
-# entry method
+# entry methods
 function collecttoplevelitems(mod::Module, path::String, text::String)
   pathitemsmaps = PathItemsMaps()
   return if mod == Main || isuntitled(path)
     # for `Main` module and unsaved editors, always use CSTPraser-based approach
     # with a given buffer text, and don't check module validity
     _collecttoplevelitems!(nothing, path, text, pathitemsmaps)
   else
-    _collecttoplevelitems!(mod, pathitemsmaps)
+    collecttoplevelitems!(mod, pathitemsmaps)
   end
 end
-
-# entry method when called from docpane/workspace
-function collecttoplevelitems(mod::Module, path::Nothing, text::String)
+function collecttoplevelitems(mod::Module, path::Nothing, text::String) # when called from docpane/workspace
   pathitemsmaps = PathItemsMaps()
-  _collecttoplevelitems!(mod, pathitemsmaps)
+  collecttoplevelitems!(mod, pathitemsmaps)
 end
 
 # sub entry method
-function _collecttoplevelitems!(mod::Module, pathitemsmaps::PathItemsMaps)
+function collecttoplevelitems!(mod::Module, pathitemsmaps::PathItemsMaps)
   entrypath, paths = modulefiles(mod)
   return if entrypath !== nothing # Revise-like approach
     _collecttoplevelitems!(stripdotprefixes(string(mod)), entrypath, paths, pathitemsmaps)