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

Future API changes #56

Open
1 of 3 tasks
ljedrz opened this issue Jul 23, 2024 · 0 comments
Open
1 of 3 tasks

Future API changes #56

ljedrz opened this issue Jul 23, 2024 · 0 comments

Comments

@ljedrz
Copy link
Owner

ljedrz commented Jul 23, 2024

With the latest PR I've noticed one potential improvement that is unlikely to ever be truly needed, but would make the library more airtight; unfortunately it would break the API, so it will need to wait for a 4.0 release. Therefore, I'm opening this issue to list any such potential future changes, and I'll be collecting them into a dedicated branch.

Breaking changes (a checkmark indicates that the 4.0 branch already contains the item):

  • return usize from Term::max_depth
  • check the API guidelines (this library is quite old by now 👴); I'm sure at least C-CONV is missing
  • possibly const-ify some of the functions

Cc @AgentElement if there are any other breaking changes you'd like to see in the future or if you'd like to contribute any of the above

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

No branches or pull requests

1 participant