bonsai/harakit
5
4
Fork 3
Code Issues 37 Pull Requests 7 Projects 1 Releases 21 Activity

dj(1) segfault fix #134

Open
trinity wants to merge 30 commits from dj into main
pull from: dj
merge into: bonsai:main
Conversation 2 Commits 30 Files Changed 2 +226 -213
trinity commented 2024-07-03 20:24:19 +00:00
Owner
Copy Link
  • dj(1) is now littered with assertions for future bug report help
  • fixes #112
  • "hard" skips and seeks (without lseek(3p)) are reported in the diagnostics
  • considerable refactoring
  • updated docs
+ dj(1) is now littered with assertions for future bug report help + fixes #112 + "hard" skips and seeks (without lseek(3p)) are reported in the diagnostics + considerable refactoring + updated docs
trinity added the
bug
 label 2024-07-03 20:24:19 +00:00
trinity added 2 commits 2024-07-03 20:24:20 +00:00
trinity added 2 commits 2024-07-03 20:50:58 +00:00
trinity commented 2024-07-03 20:53:05 +00:00
Author
Owner
Copy Link

@emma could you verify #112 is fixed on this branch?

@emma could you verify #112 is fixed on this branch?
trinity requested review from emma 2024-07-03 20:53:16 +00:00
trinity requested review from silt 2024-07-03 20:53:16 +00:00
trinity changed title from WIP: dj(1) segfault fix to dj(1) segfault fix 2024-07-03 20:53:24 +00:00
emma approved these changes 2024-07-03 21:36:16 +00:00
trinity added 5 commits 2024-07-03 22:13:29 +00:00
trinity requested review from emma 2024-07-03 22:14:37 +00:00
emma requested changes 2024-07-03 22:22:19 +00:00
docs/dj.1 Outdated
@ -36,3 +36,3 @@
When seeking or skipping to a byte, writing or reading starts at the byte
immediately subsequent to the specified byte.
immediately subsequent to the specified byte. Seeks and skips aren\(cqt counted
emma commented 2024-07-03 22:21:32 +00:00
Owner
Copy Link

Isn’t this not the case anymore? I thought we write at the byte specified, now.

Isn’t this not the case anymore? I thought we write at the byte specified, now.
trinity marked this conversation as resolved
trinity added 3 commits 2024-07-04 00:31:39 +00:00
trinity requested review from emma 2024-07-04 00:31:57 +00:00
trinity added 1 commit 2024-07-04 00:44:54 +00:00
trinity added 2 commits 2024-07-04 01:07:44 +00:00
trinity added 1 commit 2024-07-04 01:22:51 +00:00
trinity added 1 commit 2024-07-04 02:52:52 +00:00
trinity added 5 commits 2024-07-05 01:46:02 +00:00
trinity added 7 commits 2024-07-05 03:32:29 +00:00
trinity added 1 commit 2024-07-05 14:06:52 +00:00
trinity commented 2024-07-05 14:17:37 +00:00
Author
Owner
Copy Link

I think this is the best dj(1) can be, at least until we spot another bug. #94 will let us move out that parse() routine which will be nice.

I'm happy with the code size, especially relative to functionality which is much improved. Many of the little nits have been picked.

I think this is the best dj(1) can be, at least until we spot another bug. #94 will let us move out that parse() routine which will be nice. I'm happy with the code size, especially relative to functionality which is much improved. Many of the little nits have been picked.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin dj:dj
git checkout dj

Merge

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff dj
git checkout main
git merge --ff-only dj
git checkout dj
git rebase main
git checkout main
git merge --no-ff dj
git checkout main
git merge --squash dj
git checkout main
git merge --ff-only dj
git checkout main
git merge dj
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
silt
emma
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
joke

you are silly!

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
joke
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: bonsai/harakit#134
No description provided.
Delete Branch "dj"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?