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

internal_repr API convention #47

Open
vlopes11 opened this issue Jan 21, 2021 · 0 comments
Open

internal_repr API convention #47

vlopes11 opened this issue Jan 21, 2021 · 0 comments
Labels
status:minor Low priority improvements team:Core Low Level Core Development Team (Rust)

Comments

@vlopes11
Copy link

Currently we rely on internal_repr of some structures to get the internal representation of the type. Example:

pub const fn internal_repr(&self) -> &[u64; 6] {

This is undesirable because it is not according to the Rust API standards. We should discuss a convention and replicate it to our stack.

@vlopes11 vlopes11 added team:Core Low Level Core Development Team (Rust) type:question labels Jan 21, 2021
@ZER0 ZER0 added the status:minor Low priority improvements label May 17, 2021
@autholykos autholykos added team:R&D Research & Development (Cryptographic Protocol) and removed team:Core Low Level Core Development Team (Rust) labels Jul 1, 2021
@autholykos autholykos added team:Core Low Level Core Development Team (Rust) and removed team:R&D Research & Development (Cryptographic Protocol) labels Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:minor Low priority improvements team:Core Low Level Core Development Team (Rust)
Projects
None yet
Development

No branches or pull requests

3 participants