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.

Sign up for GitHub

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

Jump to bottom

Consistent member variable naming #32

Open
dtebbs opened this issue Oct 21, 2021 · 1 comment
Open

Consistent member variable naming #32

dtebbs opened this issue Oct 21, 2021 · 1 comment

Comments

@dtebbs
Copy link
Contributor

dtebbs commented Oct 21, 2021

We currently have:

  • member_variable in most old code
  • member_variable_ in a few bits of old code
  • _member_variable used in most new code

Determine a policy (e.g. _member_variable for any new or edited code?) and apply it to all related repos:

  • clearmatics/zeth
  • clearmatics/zecale
  • clearmatics/libsnark
  • clearmatics/libff
  • clearmatics/libfqfft
@dtebbs dtebbs mentioned this issue Oct 21, 2021
Merged
@AntoineRondelet
Copy link
Contributor

See: #29 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dtebbs @AntoineRondelet