Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Customizable default typedef naming format #70

Closed
DBLouis opened this issue Jun 29, 2021 · 2 comments
Closed

Customizable default typedef naming format #70

DBLouis opened this issue Jun 29, 2021 · 2 comments

Comments

@DBLouis
Copy link
Contributor

DBLouis commented Jun 29, 2021

I feel like generated typedef should be either foobar_t or Foobar. Foobar_t seems wrong, but that's just my opinion. I understand that the ability to rename types is planned but in the meantime maybe the default format could be altered.

@danielhenrymantilla danielhenrymantilla changed the title Default typedef naming format Idea: customizable default typedef naming format Jun 29, 2021
@danielhenrymantilla
Copy link
Collaborator

danielhenrymantilla commented Jun 29, 2021

Hey, I agree that the choice is not optimal, but I'll go even further and say that no default choice shall suit everybody (the choice of the current default, for instance, was driven by some other tools such as SWIG and whatnot that struggle with non-canonical C, whereby they expect struct type_name and/or typedef … type_name_t around).

You have thus made me realize that even if ad-hoc renaming is on the "soonish timeline", a way to set up / override, globally, the naming scheme would be appreciated as well.

@danielhenrymantilla danielhenrymantilla changed the title Idea: customizable default typedef naming format Customizable default typedef naming format Jun 29, 2021
@danielhenrymantilla
Copy link
Collaborator

Superseded by #93

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants