Skip to content

Conversation

@quickfur
Copy link
Member

And while we're at it, might as well use the new backtick notation for $(D ...).

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @quickfur!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

will call $(D destroy) against the payload and call $(D free) to
deallocate the store. If the $(D T) payload contains any references
to GC-allocated memory, then $(RefCounted) will add it to the GC memory
to GC-allocated memory, then `RefCounted` will add it to the GC memory
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not fixing it properly (how it probably was intended): $(LREF RefCounted) ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is in the doc for RefCounted

@dlang-bot dlang-bot merged commit 3a8d144 into dlang:master Jun 29, 2017
@quickfur quickfur deleted the refcounted-ddoc branch June 29, 2017 19:21
@wilzbach
Copy link
Contributor

Let's avoid such Ddoc "bugs" in the future: #5526 and dlang/dlang.org#1782

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants