j'ai cringe

Reviewed-on: starnakin/IronGOLEM#5

src/aton.🗿

@@ -1,18 +1,16 @@
 aton(str)
 {
-	local i = 0;
 	local out = 0;
 
 	loop {
-		if ([str + i] != '+')
+		if ([str] != '+')
 			break;
-		i++;
+		str++;
 	}
 	loop {
-		if ([str + i] == 0 | isdigit([str + i]) == 0)
-			break;
-		out = out * 10 + [str + i] - '0';
-		i++;
+		if ([str] == 0 | isdigit([str]) == 0)
+			return out;
+		out = out * 10 + [str] - '0';
+		str++;
 	}
-	return out;
 }

src/aton_s.🗿

@@ -1,23 +1,19 @@
 aton_s(str)
 {
-	local i = 0;
 	local sign = 0;
 	local out = 0;
 
 	loop {
-		if ([str + i] == '-')
+		if ([str] == '-')
 			sign = sign == 0;
-		else if ([str + i] != '+')
+		else if ([str] != '+')
 			break;
-		i++;
+		str++;
 	}
 	loop {
-		if ([str + i] == 0 | isdigit([str + i]) == 0)
-			break;
-		out = out * 10 + [str + i] - '0';
-		i++;
+		if ([str] == 0 | isdigit([str]) == 0)
+			return (sign * (0 - out)) | ((sign == 0) * out);
+		out = out * 10 + [str] - '0';
+		str++;
 	}
-	if (sign)
-		return (0 - out);
-	return out;
 }

src/bzero.🗿

@@ -1 +1 @@
-bzero(tab, size) memset(tab, size, 0);
+bzero(tab, size) => memset(tab, size, 0);

src/contain_only.🗿

@@ -1,14 +1,11 @@
 contain_only(to_big, to_find)
 {
-	local tmp;
-
-	tmp = to_big;
 	loop
 	{
-		if ([tmp] == 0)
+		if ([to_big] == 0)
 			return 1;
-		if (strchr(to_find, [tmp]) == 0)
+		if (strchr(to_find, [to_big]) == 0)
 			return 0;
-		tmp++;
+		to_big++;
 	}
 }

src/free_tab.🗿

@@ -9,5 +9,5 @@ free_tab(tab)
 		free([tmp]);
 		tmp++;
 	}
-	free(tab);
+	return free(tab);
 }

src/galloc.🗿

@@ -1,4 +1,4 @@
-define HEAP_SIZE = 0x8000;
+define HEAP_SIZE = 0x4000;
 global heap[HEAP_SIZE] = 0;
 
 define PADDING_SIZE = 4;
@@ -26,11 +26,7 @@ define HEADER_SIZE = 5;
 🗿 Is used to check invalid write
 🗿 If a case doesn't equal to 0 it is an invalid write
 
-define LOCATION_INITIALISED = 0;
-define LOCATION_USED = 1;
-define LOCATION_SIZE = 2;
-define LOCATION_PREV = 3;
-define LOCATION_NEXT = 4;
+enum LOCATION_INITIALISED, LOCATION_USED, LOCATION_SIZE, LOCATION_PREV, LOCATION_NEXT;
 define LOCATION_DATA = HEADER_SIZE + PADDING_SIZE;
 
 galloc_setup_header(ptr, used, size, next_block, prev_block)

src/geadline.🗿

@@ -1,18 +1,35 @@
-geadline(prompt) {
+geadline2(prompt, text)
+{
 	local capacity = 64,
 	      size = 0,
 	      i = 0,
 	      c,
 	      a,
-	      buf = galloc(capacity);
+	      buf;
+	if (text) {
+		size = strlen(text);
+		i = size;
+		loop {
+			if (capacity > size)
+				break;
+			capacity = capacity * 2;
+		}
+		buf = galloc(capacity);
+		if (buf == NULL)
+			return NULL;
+		strcpy(buf, text);
+	} else {
+		buf = galloc(capacity);
+		if (buf == NULL)
+			return NULL;
+		[buf] = 0;
+	}
 	if (prompt)
 		putstr(prompt);
-	if (buf == NULL)
-		return NULL;
-	[buf] = 0;
+	putstr(buf);
 
 	loop {
-		red &c;
+		c = getchar();
 		if ((c == 0xffff) | (c == 0x04)) {
 			if ((size == 0) | (c == 0xffff)) {
 				free(buf);
@@ -20,8 +37,8 @@ geadline(prompt) {
 			}
 		} else if (c == 0x1b) {
 			// ESC code
-			red &c; // skip [
-			red &c; // value
+			getchar(); // skip [
+			c = getchar(); // value
 
 			if ((c == 'A') & (size > 0)) {
 				loop {
@@ -72,6 +89,9 @@ geadline(prompt) {
 					esccode('D');
 				}
 			}
+		} else if (c == '\n') {
+			wrt '\n';
+			return buf;
 		} else {
 			size = size + 1;
 			if (size >= capacity) {
@@ -98,13 +118,14 @@ geadline(prompt) {
 				esccode('D');
 			}
 			i = i + 1;
-			if (c == '\n')
-				return buf;
 		}
 	}
 }
 
-esccode(c) {
+geadline(prompt) => geadline2(prompt, NULL);
+
+esccode(c)
+{
 	wrt 0x1b;
 	wrt '[';
 	wrt c;

src/getchar.🗿

@@ -0,0 +1,6 @@
+getchar()
+{
+	local c;
+	red &c;
+	return c;
+}

src/isalpha.🗿

@@ -1 +1 @@
-isalpha(c) return (c >= 'a' & c <= 'z') | (c >= 'A' & c <= 'Z');
+isalpha(c) => (c >= 'a' & c <= 'z') | (c >= 'A' & c <= 'Z');

src/isascii.🗿

@@ -1 +1 @@
-isascii(c) return c < 128;
+isascii(c) => c < 128;

src/isdigit.🗿

@@ -1 +1 @@
-isdigit(c) return c >= '0' & c <= '9';
+isdigit(c) => c >= '0' & c <= '9';

src/isnum.🗿

@@ -1 +1 @@
-isalnum(c) return isalpha(c) | isdigit(c);
+isalnum(c) => isalpha(c) | isdigit(c);

src/isprint.🗿

@@ -1 +1 @@
-isprint(c) return c >= ' ' & c <= '~';
+isprint(c) => c >= ' ' & c <= '~';

src/memset.🗿

@@ -5,6 +5,7 @@ memset(tab, size, value)
 	loop {
 		if (i == size)
 			return (tab);
-		[tab + i++] = value;
+		[tab + i] = value;
+		i++;
 	}
 }

src/ntoa.🗿

@@ -3,7 +3,7 @@ ntoa_get_size(number)
 	local size = 0;
 	
 	if (number == 0)
-		size++;
+		return 1;
 	loop 
 	{
 		if (number == 0)
@@ -29,8 +29,8 @@ ntoa(number)
 	{
 		if (number == 0)
 			return (str);
-		[str + size - 1] = number % 10 + '0';
-		number = number / 10;
 		size--;
+		[str + size] = number % 10 + '0';
+		number = number / 10;
 	}
 }

src/ntoa_s.🗿

@@ -1,18 +1,3 @@
-ntoa_get_size(number)
-{
-	local size = 0;
-	
-	if (number == 0)
-		size++;
-	loop 
-	{
-		if (number == 0)
-			return (size);
-		number = number / 10;
-		size++;
-	}
-}
-
 ntoa_s(number)
 {
 	local str, sign, size;

src/print_raw_bit.🗿

@@ -1,6 +1,6 @@
 print_raw_bit(number)
 {
-	local tab = get_raw_bit(number), i = 0;
+	local tab = get_raw_bit(number);
 	puttab_num(tab, 16);
 	free(tab);
 }

src/putchar.🗿

@@ -1 +1,5 @@
-putchar(c) wrt c;
+putchar(c)
+{
+	wrt c;
+	return c;
+}

src/putnum.🗿

@@ -1,9 +1,6 @@
 putnum(number)
 {
-	local str;
-	str = ntoa(number);
-	if (str == 0)
-		return;
+	local str = ntoa(number);
 	putstr(str);
 	free(str);
 }

src/putstr.🗿

@@ -1,10 +1,14 @@
 putstr(str)
 {
-	local i = 0;
+	local tmp = str;
+	if (str == NULL) {
+		putstr("(null)");
+		return NULL;
+	}
 	loop {
-		if ([str + i] == 0)
-			return;
-		putchar([str + i]);
-		i++;
+		if ([tmp] == 0)
+			return str;
+		putchar([tmp]);
+		tmp++;
 	}
 }

src/puttab.🗿

@@ -1,20 +1,19 @@
 puttab_str(tab)
 {
-	local tmp = tab;
-
 	putchar('[');
 	loop
 	{
-		if ([tmp] == 0)
-			break;
+		if ([tab] == 0) {
+			putchar(']');
+			return;
+		}
 		putchar('"');
-		putstr([tmp]);
+		putstr([tab]);
 		putchar('"');
-		if ([tmp + 1] != 0)
+		tab++;
+		if ([tab] != 0)
 			putstr(", ");
-		tmp++;
 	}
-	putchar(']');
 }
 
 puttab_num(tab, size)
@@ -24,12 +23,13 @@ puttab_num(tab, size)
 	putchar('[');
 	loop
 	{
-		if (i == size)
-			break;
+		if (i == size) {
+			putchar(']');
+			return;
+		}
 		putnum([tab + i]);
 		i++;
 		if (i != size)
 			putstr(", ");
 	}
-	putchar(']');
 }

src/replace_index.🗿

@@ -1,4 +1,4 @@
-replace(str, fill, start, stop)
+replace_index(str, fill, start, stop)
 {
 	local out;
 	local sum;

src/strchri.🗿

@@ -1,12 +0,0 @@
-strchri(str, c)
-{
-	local i = 0;
-
-	loop {
-		if ([str + i] == c)
-			return (i);
-		if ([str + i] == 0)
-			return (0 - 1);
-		i++;
-	}
-}

test.sh

@@ -6,7 +6,12 @@ tester()
 		cat src/*.🗿 tests/$val.🗿 tests/test.🗿 >tmp.🗿
 		golemc tmp.🗿 > tmp.asm
 		orgaasm tmp.asm tmp.rom
-		orgaemu tmp.rom
+		if [ -f tests/$val.input ]
+		then
+			orgaemu tmp.rom < tests/$val.input
+		else
+			orgaemu tmp.rom
+		fi
 		echo
 	done
 }

tests/geadline.input

@@ -0,0 +1,3 @@
+yo
+bozo
+zboo

tests/geadline.🗿

@@ -0,0 +1,18 @@
+main()
+{
+	local ptr;
+
+	name = "geadline";
+
+	ptr = geadline("");
+	test_str(ptr, "yo", "");
+	
+	ptr = geadline("");
+	test_str(ptr, "bozo", "");
+	
+	ptr = geadline("");
+	test_str(ptr, "bozo", "arrow");
+	
+	ptr = geadline("");
+	test_num(ptr, 0, "");
+}

tests/reallocarray.🗿

@@ -1,11 +0,0 @@
-main()
-{
-	local tmp;
-	name = "reallocarray";
-
-	tmp = strdup("yo");
-	if (tmp == NULL)
-		return 1;
-	tmp = reallocarray(tmp, strlen(tmp), 5);
-	test_str(tmp, "yo", "");
-}

tests/replace.🗿

@@ -1,9 +0,0 @@
-main()
-{
-	name = "replace";
-
-	test_str(replace("yo ca va ?", "t", 2, 3), "yotca va ?", "");
-	test_str(replace("yo ca va ?", "", 2, 3), "yoca va ?", "empty fill");
-	test_str(replace("yo ca va ?", "aaaaa", 2, 3), "yoaaaaaca va ?", "");
-	test_str(replace("", "aaaaa", 0, 0), "aaaaa", "");
-}

tests/replace_index.🗿

@@ -0,0 +1,9 @@
+main()
+{
+	name = "replace_index";
+
+	test_str(replace_index("yo ca va ?", "t", 2, 3), "yotca va ?", "");
+	test_str(replace_index("yo ca va ?", "", 2, 3), "yoca va ?", "empty fill");
+	test_str(replace_index("yo ca va ?", "aaaaa", 2, 3), "yoaaaaaca va ?", "");
+	test_str(replace_index("", "aaaaa", 0, 0), "aaaaa", "");
+}

tests/strchri.🗿

@@ -1,8 +0,0 @@
-main()
-{
-	name = "strchri";
-
-	test_num(strchri("bozoman", 'm'), 4, "");
-	test_num(strchri("bozoman", 'v'), 0 - 1, "");
-	test_num(strchri("", 'v'), 0 - 1, "");
-}

wiki/strcat.md

@@ -0,0 +1,14 @@
+# STRCAT
+Strcat (string concatenate) is a function that takes two chars lists as a parameter and write the second at this end of the first (like strcat in C)
+
+## params
+1. chars list
+2. chars list
+
+## example
+```
+strcat("y", "o") "y" => "yo"
+strcat("y", "") "y" => "y"
+strcat("", "o") "" => "o"
+strcat("hello ", "world!") => "hello world!"
+```

wiki/strcpy.md

@@ -0,0 +1,14 @@
+# STRCPY
+Strcpy (string copy) is a function that takes two chars lists as a parameter and write the second in the first (like strcpy in C)
+
+## params
+1. chars list
+2. chars list
+
+## example
+```
+strcpy("y", "o") "y" => "o"
+strcpy("y", "") "y" => ""
+strcpy("", "o") "" => "o"
+strcpy("hello ", "world!") => "world!"
+```

wiki/strlen.md

@@ -0,0 +1,16 @@
+# STRLEN
+Strlen (string length) is a function that takes an chars list as a parameter and return an length (like strlen in C)
+
+## params
+1. char list
+
+## return
+number
+
+## example
+```
+strlen("ab") => 2
+strlen(NULL) => 0
+strlen("") => 0
+strlen("j'ai les cramptés") => 17
+```