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

linenoise.cpp refactoring #11301

Merged
merged 1 commit into from
Jan 21, 2025
Merged

linenoise.cpp refactoring #11301

merged 1 commit into from
Jan 21, 2025

Conversation

ericcurtin
Copy link
Collaborator

More RAII mainly

@ericcurtin ericcurtin force-pushed the linenoise.cpp2 branch 2 times, most recently from b98e210 to 1d187c0 Compare January 19, 2025 18:27
More RAII mainly

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
@ericcurtin
Copy link
Collaborator Author

This is good for review @ggerganov @slaren

Copy link
Member

@ggerganov ggerganov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't looked at the changes, just approving so you can merge.

@ericcurtin ericcurtin merged commit 2e2f8f0 into master Jan 21, 2025
46 checks passed
@ericcurtin ericcurtin deleted the linenoise.cpp2 branch January 21, 2025 09:32
anagri pushed a commit to BodhiSearch/llama.cpp that referenced this pull request Jan 26, 2025
More RAII mainly

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
tinglou pushed a commit to tinglou/llama.cpp that referenced this pull request Feb 13, 2025
More RAII mainly

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Feb 26, 2025
More RAII mainly

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
mglambda pushed a commit to mglambda/llama.cpp that referenced this pull request Mar 8, 2025
More RAII mainly

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants