hdrgen: remove trailing whitespace on attribute declarations

Fixes Issue 22353. Signed-off-by: Luís Ferreira <contact@lsferreira.net>
dlang · Oct 5, 2021 · 5394ae6 · 5394ae6
1 parent 72c288b
commit 5394ae6
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/src/dmd/hdrgen.d b/src/dmd/hdrgen.d
@@ -909,6 +909,12 @@ public:
 
     override void visit(AttribDeclaration d)
     {
+        bool hasSTC;
+        if (auto stcd = d.isStorageClassDeclaration)
+        {
+            hasSTC = stcToBuffer(buf, stcd.stc);
+        }
+
         if (!d.decl)
         {
             buf.writeByte(';');
@@ -918,10 +924,12 @@ public:
         if (d.decl.dim == 0 || (hgs.hdrgen && d.decl.dim == 1 && (*d.decl)[0].isUnitTestDeclaration()))
         {
             // hack for bugzilla 8081
+            if (hasSTC) buf.writeByte(' ');
             buf.writestring("{}");
         }
         else if (d.decl.dim == 1)
         {
+            if (hasSTC) buf.writeByte(' ');
             (*d.decl)[0].accept(this);
             return;
         }
@@ -941,8 +949,6 @@ public:
 
     override void visit(StorageClassDeclaration d)
     {
-        if (stcToBuffer(buf, d.stc))
-            buf.writeByte(' ');
         visit(cast(AttribDeclaration)d);
     }
 

diff --git a/test/compilable/extra-files/header1.di b/test/compilable/extra-files/header1.di
@@ -511,7 +511,7 @@ ref @safe int foo(return ref int a);
 ref @safe int* foo(return ref scope int* a);
 struct SafeS
 {
-	@safe 
+	@safe
 	{
 		ref SafeS foo() return;
 		scope SafeS foo2() return;

diff --git a/test/compilable/extra-files/header1i.di b/test/compilable/extra-files/header1i.di
@@ -647,7 +647,7 @@ ref @safe int* foo(return ref scope int* a)
 }
 struct SafeS
 {
-	@safe 
+	@safe
 	{
 		ref SafeS foo() return
 		{