tree 6668e6698345ab13172a6ea9315f9a458d895360
parent 197e3b695776474cea3ca2d8e365ed4e02741803
author Jonathan Amsterdam <jba@google.com> 1601503128 -0400
committer Jonathan Amsterdam <jba@google.com> 1601553253 +0000

internal/godoc: add render test, fix node removal

Add a test that verifies that removeUnusedASTNodes doesn't
change the doc.

Fix some issues with removeUnusedASTNodes: it can't remove function
bodies that may be part of examples, and it can't remove the top-level
Comments field.

Also, two changes that prevent clobbering the AST too early:

- Remove cycles from a Package when encoding, not when creating; otherwise
  we can't use the Package to render doc before we encode it.

- Tell go/doc to preserve the AST, so we can render, then encode.

Change-Id: I2725d278572cd3de3f2019157faf224ab4b592a1
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/258424
Reviewed-by: Julie Qiu <julie@golang.org>
Trust: Jonathan Amsterdam <jba@google.com>