flac icon indicating copy to clipboard operation
flac copied to clipboard

Would like to be able to forward declare struct FLAC__StreamMetadata in my own code

Open sudo-rm-rf-oops opened this issue 3 years ago • 0 comments

In file include/FLAC/format.h: we have:

typedef struct {
...
} FLAC__StreamMetadata;

Could this please be changed to read:

typedef struct FLAC__StreamMetadata {
               ^^^^^^^^^^^^^^^^^^^^
...
} FLAC__StreamMetadata;

This has the advantage that you can forward declare struct FLAC__StreamMetadata; (thus avoiding the need to include format.h) in compilation units that only need to pass pointers around. I'm not aware of any downsides. Thanks.

Appreciate the work you're doing to keep this project moving forward, BTW. Good work for no pay.

sudo-rm-rf-oops avatar Aug 01 '22 22:08 sudo-rm-rf-oops