Sebastian Wilzbach
Sebastian Wilzbach
@BBasile I can have a look at it, but I will continue to be away from my workstation for at least another week. Sorry.
> __typeid __ is reserved for compiler internals like auto-generated variables. For runtime template hooks we typically use `_d_`.
> They should point to different parts in the same file. But only if `GLINK2` or `DDLINK` was used. All the occurrences where the `.html` is linked directly, don't work...
AFAICT the links were already not working before this change, so it doesn't change the status quo. @CyberShadow okay with this change?
> I don't see the point of this change. It's part of a general cleanup with the aim of encapsulation related things under a directory, because: - it's really off-putting...
Just sth to keep in mind: Some analyzers currently emit multiple keys/messages types.
There might be a performance penalty of traversing the tree and aggregating the same information 4x than in one pass. Also they are very often pretty related.
> One thought comes to mind, would we need to support the case where a developer doesn't want the return value of their expression to be printed? Then he can...
Let's at least partially revert #303, s.t. we aren't limited by it for the future direction of this PR (-> https://github.com/dlang/tools/pull/318).
@WebDrake no worries. This is moving nowhere unfortunately for now because the logic to insert a `writeln` by default is too simplistic. FWIW I doubt that - if it works...