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

Improve naming of builtin function parameters #858

Open
fpottbaecker opened this issue Jan 6, 2024 · 0 comments
Open

Improve naming of builtin function parameters #858

fpottbaecker opened this issue Jan 6, 2024 · 0 comments
Labels
P: Builtins Package exposing Candy's compiler builtins P: Compiler: Frontend Package: The compiler frontend P: Compiler: Inkwell Backend Package: Inkwell Compiler Backend producing native code via LLVM P: Compiler: VM Package: The VM executing Candy byte code (LIR) T: Feature Type: New Features T: Refactor Type: Refactoring

Comments

@fpottbaecker
Copy link
Collaborator

Currently, functions like equals call their parameters a and b, which is applied to the whole codebase for consistency. We could use more descriptive names, such as lhs and rhs or left or right.
@fpottbaecker fpottbaecker added the T: Feature Type: New Features label Jan 6, 2024
@JonasWanke JonasWanke added P: Compiler: Frontend Package: The compiler frontend P: Compiler: VM Package: The VM executing Candy byte code (LIR) P: Builtins Package exposing Candy's compiler builtins P: Compiler: Inkwell Backend Package: Inkwell Compiler Backend producing native code via LLVM labels Jan 7, 2024
@JonasWanke JonasWanke added the T: Refactor Type: Refactoring label Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P: Builtins Package exposing Candy's compiler builtins P: Compiler: Frontend Package: The compiler frontend P: Compiler: Inkwell Backend Package: Inkwell Compiler Backend producing native code via LLVM P: Compiler: VM Package: The VM executing Candy byte code (LIR) T: Feature Type: New Features T: Refactor Type: Refactoring
Projects
🍭 Candy
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fpottbaecker @JonasWanke