Add Kym parsing support for array literals

source/coral/io.zig

@@ -143,10 +143,6 @@ pub fn allocate_many(comptime Type: type, amount: usize, allocator: Allocator) A
 		@compileError("Cannot allocate memory for 0-byte type " ++ @typeName(Type));
 	}
 
-	if (amount == 0) {
-		return &.{};
-	}
-
 	return @ptrCast([*]Type, @alignCast(@alignOf(Type), allocator.invoke(.{.size = @sizeOf(Type) * amount}) orelse {
 		return error.OutOfMemory;
 	}))[0 .. amount];

source/coral/list.zig

@@ -144,7 +144,7 @@ pub fn Stack(comptime Value: type) type {
 			const new_length = self.values.len + values.len;
 
 			if (new_length >= self.capacity) {
-				try self.grow(allocator, math.min(new_length, self.capacity));
+				try self.grow(allocator, values.len + values.len);
 			}
 
 			const offset_index = self.values.len;
@@ -170,7 +170,7 @@ pub fn Stack(comptime Value: type) type {
 			const new_length = self.values.len + amount;
 
 			if (new_length >= self.capacity) {
-				try self.grow(allocator, math.max(usize, new_length, self.capacity));
+				try self.grow(allocator, amount + amount);
 			}
 
 			const offset_index = self.values.len;

source/ona/heap.zig

@@ -7,15 +7,23 @@ const Context = struct {
 
 	const Self = @This();
 
+	const empty_allocation = [0]u8{};
+
 	fn reallocate(self: *Self, options: coral.io.AllocationOptions) ?[]u8 {
 		if (options.size == 0) {
 			if (options.allocation) |allocation| {
-				ext.SDL_free(allocation.ptr);
+				if (allocation.ptr != &empty_allocation) {
+					ext.SDL_free(allocation.ptr);
+				}
 
 				self.live_allocations -= 1;
+
+				return null;
 			}
 
-			return null;
+			self.live_allocations += 1;
+
+			return &empty_allocation;
 		}
 
 		if (options.allocation) |allocation| {

source/ona/kym/Chunk.zig

@@ -25,6 +25,7 @@ const Opcode = enum (u8) {
 	push_integer,
 	push_float,
 	push_object,
+	push_array,
 	push_table,
 
 	not,
@@ -126,19 +127,32 @@ pub fn compile_expression(self: *Self, expression: ast.Expression) types.Runtime
 			try self.emit_object(try self.intern(literal));
 		},
 
-		.table_literal => |literal| {
-			if (literal.values.len > coral.math.max_int(@typeInfo(types.Integer).Int)) {
+		.array_literal => |elements| {
+			if (elements.values.len > coral.math.max_int(@typeInfo(types.Integer).Int)) {
 				return error.OutOfMemory;
 			}
 
-			for (literal.values) |field| {
-				try self.compile_expression(field.expression.*);
+			for (elements.values) |element_expression| {
+				try self.compile_expression(element_expression);
+			}
+
+			try self.emit_opcode(.push_array);
+			try self.emit_integer(@intCast(types.Integer, elements.values.len));
+		},
+
+		.table_literal => |fields| {
+			if (fields.values.len > coral.math.max_int(@typeInfo(types.Integer).Int)) {
+				return error.OutOfMemory;
+			}
+
+			for (fields.values) |field| {
+				try self.compile_expression(field.expression);
 				try self.emit_opcode(.push_object);
 				try self.emit_object(try self.intern(field.identifier));
 			}
 
 			try self.emit_opcode(.push_table);
-			try self.emit_integer(@intCast(types.Integer, literal.values.len));
+			try self.emit_integer(@intCast(types.Integer, fields.values.len));
 		},
 
 		.binary_operation => |operation| {

source/ona/kym/ast.zig

@@ -4,6 +4,8 @@ const tokens = @import("./tokens.zig");
 
 const types = @import("./types.zig");
 
+const ArrayElements = coral.list.Stack(Expression);
+
 pub const BinaryOperation = enum {
 	addition,
 	subtraction,
@@ -33,7 +35,7 @@ pub const ParsedExpression = union (enum) {
 				};
 
 				if (tokenizer.current_token == .symbol_plus) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `+`"};
 					}
 
@@ -55,7 +57,7 @@ pub const ParsedExpression = union (enum) {
 				}
 
 				if (tokenizer.current_token == .symbol_minus) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `-`"};
 					}
 
@@ -101,7 +103,7 @@ pub const ParsedExpression = union (enum) {
 				};
 
 				if (tokenizer.current_token == .symbol_greater_than) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `>`"};
 					}
 
@@ -123,7 +125,7 @@ pub const ParsedExpression = union (enum) {
 				}
 
 				if (tokenizer.current_token == .symbol_greater_equals) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `>=`"};
 					}
 
@@ -145,7 +147,7 @@ pub const ParsedExpression = union (enum) {
 				}
 
 				if (tokenizer.current_token == .symbol_less_than) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `<`"};
 					}
 
@@ -167,7 +169,7 @@ pub const ParsedExpression = union (enum) {
 				}
 
 				if (tokenizer.current_token == .symbol_less_equals) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `<=`"};
 					}
 
@@ -213,7 +215,7 @@ pub const ParsedExpression = union (enum) {
 				};
 
 				if (tokenizer.current_token == .symbol_double_equals) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `==`"};
 					}
 
@@ -249,7 +251,7 @@ pub const ParsedExpression = union (enum) {
 
 		switch (tokenizer.current_token) {
 			.symbol_paren_left => {
-				if (!tokenizer.step()) {
+				if (!tokenizer.step(.ignore_newlines)) {
 					return ParsedExpression{.invalid = "expected an expression after `(`"};
 				}
 
@@ -263,7 +265,7 @@ pub const ParsedExpression = union (enum) {
 							expression.deinit(allocator);
 						};
 
-						if ((!tokenizer.step()) or (tokenizer.current_token != .symbol_paren_right)) {
+						if ((!tokenizer.step(.ignore_newlines)) or (tokenizer.current_token != .symbol_paren_right)) {
 							return ParsedExpression{.invalid = "expected a closing `)` after expression"};
 						}
 
@@ -278,57 +280,161 @@ pub const ParsedExpression = union (enum) {
 				}
 			},
 
-			.integer => |value| {
-				defer _ = tokenizer.step();
+			.integer => |value| return ParsedExpression{
+				.valid = .{
+					.integer_literal = coral.utf8.parse_int(
+						@typeInfo(types.Integer).Int,
+						value, .{}) catch |parse_error| {
 
-				return ParsedExpression{
-					.valid = .{
-						.integer_literal = coral.utf8.parse_int(
-							@typeInfo(types.Integer).Int,
-							value, .{}) catch |parse_error| {
-
-							return ParsedExpression{
-								.invalid = switch (parse_error) {
-									error.BadSyntax => "invalid integer literal",
-									error.IntOverflow => "integer literal is too big",
-								}
-							};
-						},
+						return ParsedExpression{
+							.invalid = switch (parse_error) {
+								error.BadSyntax => "invalid integer literal",
+								error.IntOverflow => "integer literal is too big",
+							}
+						};
 					},
-				};
+				},
 			},
 
-			.real => |value| {
-				defer _ = tokenizer.step();
+			.real => |value| return ParsedExpression{
+				.valid = .{
+					.float_literal = coral.utf8.parse_float(
+						@typeInfo(types.Float).Float,
+						value) catch |parse_error| {
 
-				return ParsedExpression{
-					.valid = .{
-						.float_literal = coral.utf8.parse_float(
-							@typeInfo(types.Float).Float,
-							value) catch |parse_error| {
+						return ParsedExpression{
+							.invalid = switch (parse_error) {
+								error.BadSyntax => "invalid float literal",
+							},
+						};
+					},
+				},
+			},
+
+			.string => |value| return ParsedExpression{
+				.valid = .{.string_literal = value},
+			},
+
+			.symbol_bracket_left => {
+				if (tokenizer.step(.ignore_newlines)) {
+					return ParsedExpression{.invalid = "unexpected end of array literal"};
+				}
+
+				var is_invalid = true;
+				var array_elements = try ArrayElements.init(allocator, 0);
+
+				defer if (is_invalid) {
+					array_elements.deinit(allocator);
+				};
+
+				while (true) {
+					switch (tokenizer.current_token) {
+						.symbol_bracket_right => {
+							_ = tokenizer.step(.ignore_newlines);
+							is_invalid = false;
 
 							return ParsedExpression{
-								.invalid = switch (parse_error) {
-									error.BadSyntax => "invalid float literal",
+								.valid = .{.array_literal = array_elements},
+							};
+						},
+
+						else => {
+							if (!tokenizer.step(.ignore_newlines)) {
+								return ParsedExpression{.invalid = "expected `]` or expression after `[`"};
+							}
+
+							var parsed_expression = try ParsedExpression.init(allocator, tokenizer);
+
+							switch (parsed_expression) {
+								.valid => |*expression| {
+									errdefer expression.deinit(allocator);
+
+									try array_elements.push_one(allocator, expression.*);
 								},
-							};
+
+								.invalid => |detail| return ParsedExpression{.invalid = detail},
+							}
 						},
-					},
-				};
+					}
+				}
 			},
 
-			.string => |value| {
-				defer _ = tokenizer.step();
+			.symbol_brace_left => {
+				if (!tokenizer.step(.ignore_newlines)) {
+					return ParsedExpression{.invalid = "unexpected end of table literal"};
+				}
 
-				return ParsedExpression{
-					.valid = .{
-						.string_literal = value,
-					},
+				var is_invalid = true;
+				var table_fields = try TableFields.init(allocator, 0);
+
+				defer if (is_invalid) {
+					table_fields.deinit(allocator);
 				};
+
+				while (true) {
+					switch (tokenizer.current_token) {
+						.symbol_brace_right => {
+							_ = tokenizer.step(.ignore_newlines);
+							is_invalid = false;
+
+							return ParsedExpression{
+								.valid = .{.table_literal = table_fields},
+							};
+						},
+
+						.local => |identifier| {
+							const key = identifier;
+
+							if (!tokenizer.step(.ignore_newlines) or tokenizer.current_token != .symbol_equals) {
+								return ParsedExpression{.invalid = "expected `=` after identifier"};
+							}
+
+							if (!tokenizer.step(.ignore_newlines)) {
+								return ParsedExpression{.invalid = "unexpected end after `=`"};
+							}
+
+							var parsed_expression = try init(allocator, tokenizer);
+
+							switch (parsed_expression) {
+								.valid => |*expression| {
+									errdefer expression.deinit(allocator);
+
+									try table_fields.push_one(allocator, .{
+										.identifier = key,
+										.expression = expression.*,
+									});
+								},
+
+								.invalid => |details| return ParsedExpression{.invalid = details},
+							}
+
+							if (!tokenizer.step(.ignore_newlines)) {
+								return ParsedExpression{.invalid = "unexpected end after expression"};
+							}
+
+							switch (tokenizer.current_token) {
+								.symbol_comma => _ = tokenizer.step(.ignore_newlines),
+
+								.symbol_brace_right => {
+									_ = tokenizer.step(.ignore_newlines);
+									is_invalid = false;
+
+									return ParsedExpression{
+										.valid = .{.table_literal = table_fields},
+									};
+								},
+
+								else => return ParsedExpression{.invalid = "expected `,` or `}` after expression"},
+							}
+						},
+
+						else => return ParsedExpression{.invalid = "expected `}` or fields in table literal"},
+					}
+				}
 			},
 
 			.symbol_minus => {
-				if (!tokenizer.step()) {
+				if (!tokenizer.step(.ignore_newlines)) {
 					return ParsedExpression{.invalid = "expected expression after numeric negation (`-`)"};
 				}
 
@@ -351,7 +457,7 @@ pub const ParsedExpression = union (enum) {
 			},
 
 			.symbol_bang => {
-				if (!tokenizer.step()) {
+				if (!tokenizer.step(.ignore_newlines)) {
 					return ParsedExpression{.invalid = "expected expression after boolean negation (`!`)"};
 				}
 
@@ -390,7 +496,7 @@ pub const ParsedExpression = union (enum) {
 				};
 
 				if (tokenizer.current_token == .symbol_asterisk) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `*`"};
 					}
 
@@ -412,7 +518,7 @@ pub const ParsedExpression = union (enum) {
 				}
 
 				if (tokenizer.current_token == .symbol_forward_slash) {
-					if (!tokenizer.step()) {
+					if (!tokenizer.step(.ignore_newlines)) {
 						return ParsedExpression{.invalid = "expected right-hand side of expression after `/`"};
 					}
 
@@ -460,16 +566,14 @@ pub const ParsedStatements = union (enum) {
 			statements_list.deinit(allocator);
 		};
 
-		while (tokenizer.step()) {
+		while (tokenizer.step(.ignore_newlines)) {
 			switch (tokenizer.current_token) {
-				.newline => {},
-
 				.keyword_return => {
 					if (has_returned) {
 						return ParsedStatements{.invalid = "cannot return more than once per function scope"};
 					}
 
-					if (tokenizer.step() and (tokenizer.current_token != .newline)) {
+					if (tokenizer.step(.include_newlines) and (tokenizer.current_token != .newline)) {
 						var parsed_expression = try ParsedExpression.init(allocator, tokenizer);
 
 						switch (parsed_expression) {
@@ -489,16 +593,14 @@ pub const ParsedStatements = union (enum) {
 						try statements_list.push_one(allocator, .return_nothing);
 					}
 
-					if (tokenizer.step() and tokenizer.current_token != .newline) {
-						return ParsedStatements{.invalid = "expected newline after expression"};
+					if (tokenizer.step(.ignore_newlines) and tokenizer.current_token != .newline) {
+						return ParsedStatements{.invalid = "expected newline"};
 					}
 
 					has_returned = true;
 				},
 
-				else => {
-					return ParsedStatements{.invalid = "invalid statement"};
-				},
+				else => return ParsedStatements{.invalid = "invalid statement"},
 			}
 		}
 
@@ -519,7 +621,8 @@ pub const Expression = union (enum) {
 	integer_literal: types.Integer,
 	float_literal: types.Float,
 	string_literal: []const u8,
-	table_literal: TableLiteral,
+	array_literal: ArrayElements,
+	table_literal: TableFields,
 	grouped_expression: *Expression,
 
 	binary_operation: struct {
@@ -533,22 +636,24 @@ pub const Expression = union (enum) {
 		expression: *Expression,
 	},
 
-	const TableLiteral = coral.list.Stack(struct {
-		identifier: []const u8,
-		expression: *Expression,
-	});
-
 	fn deinit(self: *Expression, allocator: coral.io.Allocator) void {
 		switch (self.*) {
 			.nil_literal, .true_literal, .false_literal, .integer_literal, .float_literal, .string_literal => {},
 
-			.table_literal => |*literal| {
-				for (literal.values) |field| {
-					field.expression.deinit(allocator);
-					coral.io.deallocate(allocator, field.expression);
+			.array_literal => |*elements| {
+				for (elements.values) |*element_expression| {
+					element_expression.deinit(allocator);
 				}
 
-				literal.deinit(allocator);
+				elements.deinit(allocator);
+			},
+
+			.table_literal => |*fields| {
+				for (fields.values) |*field| {
+					field.expression.deinit(allocator);
+				}
+
+				fields.deinit(allocator);
 			},
 
 			.grouped_expression => |expression| {
@@ -639,6 +744,11 @@ pub const Statements = struct {
 	}
 };
 
+const TableFields = coral.list.Stack(struct {
+	identifier: []const u8,
+	expression: Expression,
+});
+
 pub const UnaryOperation = enum {
 	boolean_negation,
 	numeric_negation,

source/ona/kym/tokens.zig

@@ -43,7 +43,6 @@ pub const Token = union(enum) {
 	pub fn text(self: Token) []const u8 {
 		return switch (self) {
 			.unknown => |unknown| @ptrCast([*]const u8, &unknown)[0 .. 1],
-			.newline => "newline",
 			.identifier_global => |identifier| identifier,
 			.identifier_local => |identifier| identifier,
 
@@ -83,22 +82,18 @@ pub const Tokenizer = struct {
 	previous_token: Token = .newline,
 	current_token: Token = .newline,
 
-	pub fn has_next(self: Tokenizer) bool {
-		return self.source.len != 0;
-	}
-
-	pub fn step(self: *Tokenizer) bool {
+	pub fn step(self: *Tokenizer, newline_rules: enum { ignore_newlines, include_newlines }) bool {
 		self.previous_token = self.current_token;
 
 		var cursor = @as(usize, 0);
 
 		defer self.source = self.source[cursor ..];
 
-		while (self.has_next()) switch (self.source[cursor]) {
+		while (cursor < self.source.len) switch (self.source[cursor]) {
 			'#' => {
 				cursor += 1;
 
-				while (self.has_next() and (self.source[cursor] == '\n')) {
+				while (cursor < self.source.len and self.source[cursor] == '\n') {
 					cursor += 1;
 				}
 			},
@@ -107,10 +102,17 @@ pub const Tokenizer = struct {
 
 			'\n' => {
 				cursor += 1;
-				self.lines_stepped += 1;
-				self.current_token = .newline;
 
-				return true;
+				switch (newline_rules) {
+					.include_newlines => {
+						self.lines_stepped += 1;
+						self.current_token = .newline;
+
+						return true;
+					},
+
+					else => {},
+				}
 			},
 
 			'0' ... '9' => {
@@ -118,13 +120,13 @@ pub const Tokenizer = struct {
 
 				cursor += 1;
 
-				while (self.has_next()) switch (self.source[cursor]) {
+				while (cursor < self.source.len) switch (self.source[cursor]) {
 					'0' ... '9' => cursor += 1,
 
 					'.' => {
 						cursor += 1;
 
-						while (self.has_next()) switch (self.source[cursor]) {
+						while (cursor < self.source.len) switch (self.source[cursor]) {
 							'0' ... '9' => cursor += 1,
 							else => break,
 						};
@@ -159,40 +161,52 @@ pub const Tokenizer = struct {
 				switch (identifier[0]) {
 					'n' => if (coral.io.ends_with(identifier, "il")) {
 						self.current_token = .keyword_nil;
+
+						return true;
 					},
 
 					'f' => if (coral.io.ends_with(identifier, "alse")) {
 						self.current_token = .keyword_false;
+
+						return true;
 					},
 
 					't' => if (coral.io.ends_with(identifier, "rue")) {
 						self.current_token = .keyword_true;
+
+						return true;
 					},
 
 					'r' => if (coral.io.ends_with(identifier, "eturn")) {
 						self.current_token = .keyword_return;
+
+						return true;
 					},
 
 					's' => if (coral.io.ends_with(identifier, "elf")) {
 						self.current_token = .keyword_self;
+
+						return true;
 					},
 
-					else => self.current_token = .{.local = identifier},
+					else => {},
 				}
 
+				self.current_token = .{.local = identifier};
+
 				return true;
 			},
 
 			'@' => {
 				cursor += 1;
 
-				if (self.has_next()) switch (self.source[cursor]) {
+				if (cursor < self.source.len) switch (self.source[cursor]) {
 					'A'...'Z', 'a'...'z', '_' => {
 						const begin = cursor;
 
 						cursor += 1;
 
-						while (self.has_next()) switch (self.source[cursor]) {
+						while (cursor < self.source.len) switch (self.source[cursor]) {
 							'0'...'9', 'A'...'Z', 'a'...'z', '_' => cursor += 1,
 							else => break,
 						};
@@ -209,7 +223,7 @@ pub const Tokenizer = struct {
 
 						cursor += 1;
 
-						while (self.has_next()) switch (self.source[cursor]) {
+						while (cursor < self.source.len) switch (self.source[cursor]) {
 							'"' => break,
 							else => cursor += 1,
 						};
@@ -235,7 +249,7 @@ pub const Tokenizer = struct {
 
 				cursor += 1;
 
-				while (self.has_next()) switch (self.source[cursor]) {
+				while (cursor < self.source.len) switch (self.source[cursor]) {
 					'"' => break,
 					else => cursor += 1,
 				};
@@ -319,7 +333,7 @@ pub const Tokenizer = struct {
 			'=' => {
 				cursor += 1;
 
-				if (self.has_next()) {
+				if (cursor < self.source.len) {
 					switch (self.source[cursor]) {
 						'=' => {
 							cursor += 1;
@@ -347,7 +361,7 @@ pub const Tokenizer = struct {
 			'<' => {
 				cursor += 1;
 
-				if (self.has_next() and (self.source[cursor] == '=')) {
+				if (cursor < self.source.len and (self.source[cursor] == '=')) {
 					cursor += 1;
 					self.current_token = .symbol_less_equals;
 
@@ -362,7 +376,7 @@ pub const Tokenizer = struct {
 			'>' => {
 				cursor += 1;
 
-				if (self.has_next() and (self.source[cursor] == '=')) {
+				if (cursor < self.source.len and (self.source[cursor] == '=')) {
 					cursor += 1;
 					self.current_token = .symbol_greater_equals;