Commit Diff

Commit:
1309450668aa571dee97f4373f9555b4fddcf1aa
From:
Fazlul Shahriar <fshahriar@gmail.com>
Via:
Dan Cross <crossd@gmail.com>
Date:
Message:
awk: split record into runes for empty FS (#292) awk was splitting records into bytes instead of runes for empty FS. For example, this was printing only the first byte of the utf-8 encoding of é: echo é | awk 'BEGIN{FS=""}{print $1}' The change just copies how the `split` function handles runes. Originally reported by kris on twitter: https://twitter.com/p9luv/status/1180436083433201665
Actions:
Patch | Tree
commit - 715807d706cd13bc583588477a84090fbf02e057
commit + 1309450668aa571dee97f4373f9555b4fddcf1aa
blob - 6a6849c54f2845c52a2ec5c4d2a8baa30d7626dc
blob + 3eb30687cf9f746445c95938fe1810886c652fc6
--- src/cmd/awk/lib.c
+++ src/cmd/awk/lib.c
@@ -29,6 +29,7 @@ THIS SOFTWARE.
 #include <errno.h>
 #include <stdlib.h>
 #include <stdarg.h>
+#include <utf.h>
 #include "awk.h"
 #include "y.tab.h"
 
@@ -293,15 +294,19 @@ void fldbld(void)	/* create fields from current record
 		}
 		*fr = 0;
 	} else if ((sep = *inputFS) == 0) {		/* new: FS="" => 1 char/field */
-		for (i = 0; *r != 0; r++) {
-			char buf[2];
+		int nb;
+		for (i = 0; *r != 0; r += nb) {
+			Rune rr;
+			char buf[UTFmax+1];
+
 			i++;
 			if (i > nfields)
 				growfldtab(i);
 			if (freeable(fldtab[i]))
 				xfree(fldtab[i]->sval);
-			buf[0] = *r;
-			buf[1] = 0;
+			nb = chartorune(&rr, r);
+			memmove(buf, r, nb);
+			buf[nb] = '\0';
 			fldtab[i]->sval = tostring(buf);
 			fldtab[i]->tval = FLD | STR;
 		}