ForNeVeR / ForNeVeR/Cesium

`TranslationUnitContext` uses empty struct as a placeholder for forward declaration

Open
#504 0 comments 0 reactions 0 assignees View on GitHub
area:compiler kind:refactor status:help-wanted
Dominant language
C#
Stars
451
Forks
49
Avg merge
13h 59m
Merged PRs (30d)
6

Description

See this snippet: https://github.com/ForNeVeR/Cesium/blob/532fbec3dcae1d5084971ab440f0ae865146d199/Cesium.CodeGen/Contexts/TranslationUnitContext.cs#L186-L196

From this, it looks like we use an empty struct (with no members) as a placeholder for a forward declaration, or something similar.

Empty structs is a useful standard extension, and we may choose to implement it later; thus this will break.

We should improve the architecture here and make the placeholder more explicit.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.