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

Rename the "init" function to avoid clashes #45

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Rename the "init" function to avoid clashes #45

wants to merge 2 commits into from

Conversation

notmgsk
Copy link
Member

@notmgsk notmgsk commented Jul 31, 2023

No description provided.

notmgsk added 2 commits July 31, 2023 14:00
@notmgsk
Change default "init" library function name to something less common
69bb316 
Reduces the chance that we'll clash with another "init" function.

Also makes the name configurable.
@notmgsk
Make libcalc Makefile useful on Linux
827d931
@notmgsk notmgsk changed the title Rename the "init" function Rename the "init" function to avoid clashes Jul 31, 2023
@karlosz
Copy link
Collaborator

karlosz commented Jul 31, 2023

init should just be called lisp_init

@stylewarning
Copy link
Member

init should just be called lisp_init

Or, when LC_ALL=en_GB.UTF-8, it should be lisp_innit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kartik-s kartik-s Awaiting requested review from kartik-s

@karlosz karlosz Awaiting requested review from karlosz

@colescott colescott Awaiting requested review from colescott

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@notmgsk @karlosz @stylewarning