Sebastian Wilzbach

Results 359 comments of 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...