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.

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

Jump to bottom

Linedotc edits #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Linedotc edits #18

wants to merge 3 commits into from

Conversation

rvillalob
Copy link

No description provided.

@rvillalob
Update utf8.c
dea9c19 
The function unicode_to_utf8 was failing to produce the correct leading byte of the UTF8 sequence because the loop was stopping one iteration too soon.
@rvillalob
Merge pull request #1 from rvillalob/utf8dotc-edits
8029baf 
Update utf8.c
@rvillalob
Update line.c
0d950ca 
If the current kill buffer chunk was just assigned NULL, shouldn't kused be assigned a value of zero?
@rfivet
Copy link

rfivet commented Oct 1, 2018

If you set kused to 0 instead of KBLOCK in kdelete, then kinsert will not allocate a chunk on next yank.
When kused reach KBLOCK, it's the trigger to allocate next chunk, so initial state is kused set as KBLOCK.
If you rewrite kdelete as you did, you need rework buffer allocation in kinsert accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rvillalob @rfivet