Applying REVERSE opcode for byte arrays too

[igormcoelho]

This is a possible solution to the issue raised at neo-vm page: #17

[erikzhang]

For Array, REVERSE does not push the result back onto the stack:

neo-vm/src/neo-vm/ExecutionEngine.cs

Lines 840 to 843 in 94a91ae

	if (arrItem is VMArray array)
	{
	array.Reverse();
	}

But in this PR, for byte array, it will push the result back onto the stack.

This makes the behavior of the REVERSE instruction inconsistent. I would prefer to add a new opcode.

Maybe we can delete the SIZE opcode because ARRAYSIZE can replace it.

neo-vm/src/neo-vm/ExecutionEngine.cs

Lines 688 to 696 in 94a91ae

	case OpCode.ARRAYSIZE:
	{
	StackItem item = context.EvaluationStack.Pop();
	if (item is ICollection collection)
	context.EvaluationStack.Push(collection.Count);
	else
	context.EvaluationStack.Push(item.GetByteArray().Length);
	}
	break;

[igormcoelho]

That's true, it's something I was starting to be worried about too... perhaps it becomes harder to simulate the results of the vm, and perhaps not worth the risk. Better not create opcodes with this irregular format.

[igormcoelho]

I'm abandoning this PR to favor this proposal here, I think it's better: #59