aionnetwork / AVM

Enabling Java code to run in a blockchain environment
https://theoan.com/
MIT License
49 stars 25 forks source link

correct 2 corrupted "t" via copy&paste, fixes #367 #368

Closed ghost closed 5 years ago

jeff-aion commented 5 years ago

This is a good catch and I am pretty surprised that this was only observed on Windows.

ghost commented 5 years ago

Totally strange, never hit on such a bug before. There was a "ghost-char" between the "t" and the (" in the two avm_wait(

(as an off-topic note: i just evaluate EOS, and am very disappointed by its low usability. The missing return-values make working with EOS-style-CPP quite inconvenient. Be aware that AVM outperforms EOS in context of convenience. Yet, you need to work on usability of tools/docs, its very important).

jeff-aion commented 5 years ago

Yeah, most tools hide it but vim does display it (even the GitHub diff doesn't show it). Reminds me of the old days of BBEdit and its highly useful "zap gremlins" feature.

We had some fairly public AVM testing, this past weekend, and that is also what we found. People were generally happy with the system but there are gaps in our docs and some rough edges on our tools. So yes, we are aware that this is an area where we need more work.

Thanks for letting us know your sense of this.

ghost commented 5 years ago

"zap gremlins" feature

haha! fits with "ghost-char"!

So yes, we are aware that this is an area where we need more work.

And you need to be aware of another thing:

This work is something that needs to done by a person which is a natural (in simplifying systems/docs), and which does not know the (AION AVM) system very well (yet).