Bug 20074 – header file generation doesn't include attributes with CallExp

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P1
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2019-07-22T22:37:11Z
Last change time
2019-08-07T11:06:14Z
Keywords
pull
Assigned to
No Owner
Creator
Hiroki Noda

Comments

Comment #0 by kubo39 — 2019-07-22T22:37:11Z
Tested dmd version: 2.079.1 / 2.087.0 Input: --- // dmd -c -Hf=hdrgenbug.di hdrgenbug.d class C { this() @safe { () @trusted { } (); } } --- Output: --- class C { @safe this() { () { } (); } } ---
Comment #1 by dlang-bot — 2019-07-24T15:50:48Z
@RazvanN7 created dlang/dmd pull request #10217 "Fix Issue 20074 - header file generation doesn't include attributes with CallExp" fixing this issue: - Fix Issue 20074 - header file generation doesn't include attributes with CallExp https://github.com/dlang/dmd/pull/10217
Comment #2 by petar.p.kirov — 2019-07-29T21:25:00Z
As discussed in the above linked PR, we're not sure if non-templated constructor bodies should be part of header generation, as it seems including them was not deliberate in the first place. Do you have use case which requires all constructors bodies to be part of the compiler generated header? Otherwise the underlying issue here is that function literal attributes were not printed.
Comment #3 by dlang-bot — 2019-08-07T11:06:14Z
dlang/dmd pull request #10217 "Fix Issue 20074 - header file generation doesn't include attributes with CallExp" was merged into master: - f8ec11f2781e6653bd42a8415603571239dee7d0 by RazvanN7: Fix Issue 20074 - header file generation doesn't include attributes with CallExp https://github.com/dlang/dmd/pull/10217