Skip to content

Add ability to paste c-source files#100

Merged
jordalgo merged 2 commits into
libbpf:masterfrom
jordalgo:c-source-paste
Nov 19, 2025
Merged

Add ability to paste c-source files#100
jordalgo merged 2 commits into
libbpf:masterfrom
jordalgo:c-source-paste

Conversation

@jordalgo

@jordalgo jordalgo commented Nov 8, 2025

Copy link
Copy Markdown
Collaborator

The pasted C source files need to match up
with what the verifier has or else we don't
allow the contents to be overwritten.

c-source-paste.mov

@theihor

theihor commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

@jordalgo in the video it looks like the indentation of the pasted C source is lost. How does that happen? I think it makes sense to preserve it in the UI.

The pasted C source files need to match up
with what the verifier has or else we don't
allow the contents to be overwritten.
@jordalgo

Copy link
Copy Markdown
Collaborator Author

@theihor Good catch. This just needed an additional css property white-space: pre; (now fixed).

@jordalgo

Copy link
Copy Markdown
Collaborator Author

Note: In a follow-up PR, I want to include the pasted C files in with the local storage log data.

This is so users don't have to re-paste C source files
when they re-load a log from local storage.
@jordalgo

Copy link
Copy Markdown
Collaborator Author

Note: In a follow-up PR, I want to include the pasted C files in with the local storage log data.

Added as a separate commit in this PR.

Comment thread src/App.tsx
@jordalgo jordalgo merged commit 7ceb074 into libbpf:master Nov 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants