Skip to content

feat: update libdave + prebuilt library#3

Merged
topi314 merged 59 commits intodisgoorg:masterfrom
davfsa:task/prebuild-libdave
Feb 11, 2026
Merged

feat: update libdave + prebuilt library#3
topi314 merged 59 commits intodisgoorg:masterfrom
davfsa:task/prebuild-libdave

Conversation

@davfsa
Copy link
Copy Markdown
Contributor

@davfsa davfsa commented Jan 18, 2026

No description provided.

Signed-off-by: davfsa <davfsa@gmail.com>
- memory leaks
- don't vendor libdave to avoid runtime loading issues
- copy C arrays over to go to allow the GC to optimize usage fully
@davfsa davfsa marked this pull request as ready for review January 20, 2026 10:41
@davfsa davfsa force-pushed the task/prebuild-libdave branch from 3aa01f5 to f4c06d7 Compare January 20, 2026 14:43
Comment thread libdave/session.go Outdated
defaultLogger.Load().Error(
C.GoString(reason),
slog.String("source", C.GoString(source)),
slog.String("authSessionID", C.GoString((*C.char)(userData))),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This needs to be passed through a handler

Comment thread scripts/libdave_install.sh Outdated
Signed-off-by: davfsa <davfsa@gmail.com>
Signed-off-by: davfsa <davfsa@gmail.com>
@topi314 topi314 merged commit 4ef3e35 into disgoorg:master Feb 11, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants