checkpoint

.github/workflows/release.yml

@@ -10,29 +10,6 @@ permissions:
   contents: write
 
 jobs:
-  goreleaser:
-    name: GoReleaser
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version-file: 'go.mod'
-          cache: true
-
-      - name: Run GoReleaser
-        uses: goreleaser/goreleaser-action@v6
-        with:
-          distribution: goreleaser
-          version: latest
-          args: release --clean
-        env:
-          GITHUB_TOKEN: ${{ secrets.GH_CPAT }}
   homebrew:
     name: Bump Homebrew formula
     # Skip this job in case of git pushes to prerelease tags
@@ -49,7 +26,7 @@ jobs:
       - uses: mislav/bump-homebrew-formula-action@v3
         with:
           formula-name: ke
-          formula-path: Formula/kw.rb
+          formula-path: Formula/ke.rb
           homebrew-tap: kisom/homebrew-tap
           base-branch: master
           commit-message: |

CMakeLists.txt

@@ -2,10 +2,11 @@ cmake_minimum_required(VERSION 3.15)
 project(ke C)  # Specify C language explicitly
 
 set(CMAKE_C_STANDARD 99)
-set(KE_VERSION "1.0.3")
+set(KE_VERSION "1.0.4")
 
 set(CMAKE_C_FLAGS "-Wall -Wextra -pedantic -Wshadow -Werror -std=c99 -g")
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -D_DEFAULT_SOURCE -D_XOPEN_SOURCE")
+set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=address -fno-omit-frame-pointer")
 
 
 # Add executable

README.txt

@@ -6,4 +6,9 @@ used fairly often.
 
 See the man page for more info.
 
+It should be available via homebrew, even:
+
+    brew tap kisom/homebrew-tap
+    brew install ke
+
 Released under an ISC license.

ke.1

@@ -24,8 +24,6 @@ saving a file can be done with either C-k s or C-k C-s.
 .Bl -tag -width xxxxxxxxxxxx -offset indent
 .It C-k BACKSPACE
 Delete from the cursor to the beginning of the line.
-.It C-k C-d
-Delete the current row.
 .It C-k d
 Delete from the cursor to the end of the line.
 .It C-k e

main.c

@@ -30,6 +30,14 @@
 #define TAB_STOP	8
 #define MSG_TIMEO	3
 
+/*
+ * Provide a sensiblerdefault version string if not supplied by the
+ * build system
+ */
+#ifndef KE_VERSION
+#define KE_VERSION "ke devel"
+#endif
+
 /*
  * define the keyboard input modes
  * normal: no special mode
@@ -50,13 +58,18 @@
 int
 next_power_of_2(int n)
 {
+	if (n < 2) {
+		n = 2;
+	}
+
 	n--;
 	n |= n >> 1;
 	n |= n >> 2;
 	n |= n >> 4;
 	n |= n >> 8;
 	n |= n >> 16;
-	return n+1;
+
+	return n + 1;
 }
 
 /*
@@ -66,11 +79,11 @@ int
 cap_growth(int cap, int sz)
 {
     if (cap == 0) {
-		return INITIAL_BUFSIZE;
+	    cap = INITIAL_BUFSIZE;
     }
 
-	while (sz < cap) {
-		cap = next_power_of_2(cap);
+    while (cap <= sz) {
+        cap = next_power_of_2(cap + 1);
     }
 
     return cap;
@@ -272,7 +285,11 @@ ab_append(struct abuf *buf, const char *s, int len)
 			buf->cap = 64;
 		}
 
-		while (sz < buf->cap) {
+		/*
+		 * grow until capacity is at least the required
+		 * size
+		 */
+		while (sz > buf->cap) {
 			if (buf->cap > INT_MAX/2) {
 				buf->cap = INT_MAX;
 				break;
@@ -367,9 +384,10 @@ erow_update(struct erow *row)
 	 * TODO(kyle): I'm not thrilled with this double-render.
 	 */
 	for (j = 0; j < row->size; j++) {
-		if (row->line[j] == '\t') {
+		unsigned char ch = (unsigned char)row->line[j];
+		if (ch == '\t') {
 			tabs++;
-		} else if (!isprint(row->line[j])) {
+		} else if (!isprint(ch)) {
 			ctrl++;
 		}
 	}
@@ -379,20 +397,21 @@ erow_update(struct erow *row)
 		row->rsize = 0;
 	}
 	row->render = NULL;
-	row->render = malloc(row->size + (tabs * (TAB_STOP-1)) + (ctrl * 3) + 1);
+	row->render = malloc(row->size + (tabs*(TAB_STOP-1)) + (ctrl*3) + 1);
 	assert(row->render != NULL);
 
 	for (j = 0; j < row->size; j++) {
-		if (row->line[j] == '\t') {
+		unsigned char ch = (unsigned char)row->line[j];
+		if (ch == '\t') {
 			do {
 				row->render[i++] = ' ';
 			} while ((i % TAB_STOP) != 0);
-		} else if (!isprint(row->line[j])) {
+		} else if (!isprint(ch)) {
 			row->render[i++] = '\\';
-			row->render[i++] = nibble_to_hex(row->line[j] >> 4);
-			row->render[i++] = nibble_to_hex(row->line[j] & 0x0f);
+			row->render[i++] = nibble_to_hex((char)(ch >> 4));
+			row->render[i++] = nibble_to_hex((char)(ch & 0x0f));
 		} else {
-			row->render[i++] = row->line[j];
+			row->render[i++] = (char)ch;
 		}
 	}
 
@@ -540,6 +559,8 @@ row_append_row(struct erow *row, char *s, int len)
 	assert(row->line != NULL);
 	memcpy(&row->line[row->size], s, len);
 	row->size += len;
+
+	row->cap = row->size + 1;
 	row->line[row->size] = '\0';
 	erow_update(row);
 	editor.dirty++;
@@ -560,8 +581,11 @@ row_insert_ch(struct erow *row, int at, int16_t c)
 	}
 	assert(c > 0);
 
-	if (row->size == row->cap) {
-		ncap = cap_growth(row->cap, row->size+1);
+    /* We will move existing bytes including the current NUL, so we need
+     * at least row->size + 2 bytes of storage (new char + NUL). Ensure
+     * capacity exceeds index row->size + 1. */
+    if (row->size + 1 >= row->cap) {
+        ncap = cap_growth(row->cap, row->size + 1);
         nline = realloc(row->line, ncap);
 
         assert(nline != NULL);
@@ -573,8 +597,6 @@ row_insert_ch(struct erow *row, int at, int16_t c)
         row->line = nline;
     }
 
-	row->line = realloc(row->line, row->size+2);
-	assert(row->line != NULL);
     memmove(&row->line[at+1], &row->line[at], row->size - at + 1);
     row->size++;
     row->line[at] = c & 0xff;
@@ -1141,6 +1163,7 @@ process_kcommand(int16_t c)
 	case 'x':
 		exit(save_file());
 	case CTRL_KEY('d'):
+	case CTRL_KEY('y'):
 		delete_row(editor.cury);
 		break;
 	case 'd':