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

Remove optionals where they don't make sense #56

Open
Guigui220D opened this issue Jul 6, 2024 · 0 comments
Open

Remove optionals where they don't make sense #56

Guigui220D opened this issue Jul 6, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@Guigui220D
Copy link
Owner

Some places I used optionals just because the translate c made signatures with optional pointers, but it seems like

  • some methods always return a pointer
  • even when it is uninitialized it returns a value that would segfault (and I can't check that)
    This would avoid some checks both inside the function and for the caller
@Guigui220D Guigui220D added the enhancement New feature or request label Jul 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant