On Mon, Feb 4, 2013 at 2:09 PM, François Garillot
<notifications@github.com>wrote:

> In src/compiler/scala/tools/nsc/interactive/Doc.scala:
>
> > -    val sig = ask { () => externalSignature(sym) }
> > -    findExternalLink(sym, sig) map { link => UrlResult(link.url) } orElse {
> > -      val resp = new Response[Tree]
> > -      // Ensure docComment tree is type-checked.
> > -      val pos = ask { () => docCommentPos(sym) }
> > -      askTypeAt(pos, resp)
> > -      resp.get.left.toOption flatMap { _ =>
> > -        ask { () =>
> > -          val comment = parseAtSymbol(expandedDocComment(sym), rawDocComment(sym), pos, Some(site))
> > + /**
> > +  * Gets the documentation either as a Comment object or as a URL.
> > +  * @param sym      The symbol whose documentation should be retrieved.
> > +  * @param site     The place where sym is observed.
> > +  * @param source   The source file to look for the symbol.
> > +  */
> > +  def retrieve(sym: Symbol, site: Symbol, source: SourceFile): Option[DocResult] = {
>
> I looked for usages of this one by grepping, and found it frustrating to
> grep for something called retrieve. Would you mind renaming it to
> something like retrieveDocForSymbol (or something expressive you'd like
> better) ?
>
Done.

>  —
> Reply to this email directly or view it on GitHub<https://github.com/scala/scala/pull/2016/files#r2874384>.
>
>