Don't remove docs in block yielding of macro

[makenowjust]

For example, we have a below code:

record Foo,
  foo : String do
  # this is cool
  def super_foo
    "Super #{foo}!"
  end
end

then run crystal doc, we get such a doc:

There are no Foo#super_foo doc comment, it is just wrong. I fixed it by adding emit_doc optional arg to ASTNode#to_s and using it in macro interpreter.

[asterite]

Good catch!

The emit_doc optional argument looks good, but the with_doc argument is not necessary: the doc property should always be cloned (copied) from the original node. This will simplify code a lot, and doesn't involve any performance or memory penalty.

[makenowjust]

@asterite Thanks! I'll fix it.

[asterite]

@makenowjust Great, thank you!! :-)

Now that I think about it, the comments are probably missing too when you declare a type inside record, or well you use record inside record with comment:

record Foo do
  # Bar comment
  record Bar

  # Baz comment
  class Baz
  end
end

This is because ClassDef, ModuleDef, EnumDef and Call also have a doc property, so this logic should probably be applied to those too. I can merge this PR and we (either me, you or somebody else) can do it later, or you can do it all in this PR? What do you prefer?

[makenowjust]

@asterite I think it is no problem. Because doc property is copied by ASTNode#clone, not Def#clone_without_location, so we can copy doc every AST nodes.

[asterite]

@makenowjust ありがとう！

[makenowjust]

@asterite Muchas gracias