Skip to content

Remove unnecessary string conversions in error message construction #5826

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

Merged
youknowone merged 10 commits into from
Jun 24, 2025
Merged

Remove unnecessary string conversions in error message construction #5826

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
e7923f6
Make `new_X_error` recv `impl Into<String>`
ShaharNaveh Jun 22, 2025
485dea2
Maks the rest of the methods to be generic
ShaharNaveh Jun 22, 2025
93de8d8
remove to_owned
ShaharNaveh Jun 22, 2025
8d7deb2
Remove `.to_owned()` from more files
ShaharNaveh Jun 23, 2025
9e83d43
Make `new_struct_error` to be generic as well
ShaharNaveh Jun 23, 2025
704cc8f
Remove `.to_owned()` from more places
ShaharNaveh Jun 23, 2025
2eefb81
Resolve wasm errors
ShaharNaveh Jun 23, 2025
628ee03
Merge remote-tracking branch 'upstream/main' into cln-msg-generic-string
ShaharNaveh Jun 24, 2025
ebc88bc
Revert changes to `new_exception_msg` and `new_exception_msg_dict`
ShaharNaveh Jun 24, 2025
324b8be
fix wasm
ShaharNaveh Jun 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'upstream/main' into cln-msg-generic-string
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Jun 24, 2025
commit 628ee03881d0d7aeef7dd07c8cdd2afd3e60318d

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.