[matt-edmondson]

Fixes #157

What changed

TemplateRendering.WriteBody decided a body's shape by counting non-blank lines. A line counted even when it was only a // comment or a #pragma directive. So // nothing to reset became void M() // nothing to reset, and two or more such lines were spliced below the declaration with no braces. In both cases the member had no body, which gives CS1002.

Now, when no line of the body holds code, the body is written as a braced block on the lines after the declaration, with the comments and directives kept inside it:

void M()
{
	// nothing to reset
}

A line holds code when its text before any line comment is non-empty and does not start with #. The comment boundary comes from the existing FindCodeEnds, the helper WriteExpressionBody already uses, so a // inside a string literal is not taken for a comment. A one-line body that does hold code, such as => x;, { } or => x; // trailing, still stays on the declaration line.

Tests

• New tests cover the four inputs from the issue: one line comment, two line comments, a comment-only ConstructorTemplate body, and an empty PragmaScope.
• AOneLineBodyWithCodeStaysOnTheDeclarationLine pins the unchanged one-line shape.
• With the fix reverted, the four new brace tests fail.
• The full suite passes: 256 tests on net10.0.

---

Generated by Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud