Operators '&&' and '||' may not be acting correctly.

[Ratstail91]

Toy/source/toy_vm.c

Lines 336 to 347 in 7398898

	if (opcode == TOY_OPCODE_AND) {
	Toy_Value right = Toy_popStack(&vm->stack);
	Toy_Value left = Toy_popStack(&vm->stack);
	Toy_pushStack(&vm->stack, TOY_VALUE_FROM_BOOLEAN( Toy_checkValueIsTruthy(left) && Toy_checkValueIsTruthy(right) ));
	}
	else if (opcode == TOY_OPCODE_OR) {
	Toy_Value right = Toy_popStack(&vm->stack);
	Toy_Value left = Toy_popStack(&vm->stack);
	Toy_pushStack(&vm->stack, TOY_VALUE_FROM_BOOLEAN( Toy_checkValueIsTruthy(left) || Toy_checkValueIsTruthy(right) ));
	}

In most languages, the && and || operators have short-circuiting behavior. As it is, the current implementation doesn't do that.

I may need the JUMP keyword to skip over the rhs if the top value is true.

[Ratstail91]

Formal definition for &&:

Return the first falsy value, or the last value, skipping the evaluation of other operands.

Formal definition for ||:

Return the first truthy value, or the last value, skipping the evaluation of other operands.

This definition matches JavaScript's implementation.