Rewrite Accessor to fix array access issues, also add a Delete method

[Niels-Be]

Summary

This is a rewrite of the accessor code to fix a lot of issues with access to arrays.

I added some tests that were failing bevor this rewrite.

There should be no breaking changes with this code. However the new code could allow errors to be forwarded to the caller. e.g. for Get() to differentiate between not found and json null value. Or to notify the caller about an invalid path expression.

Maybe add new methods for this?

In this PR all errors are just swallowed to keep compatibility.

Additionally a new Delete method (as requested in #141) is introduced which was trivial to add with the new code.

Checklist

• [x] Tests are passing: task test
• [x] Code style is correct: task lint

[hanzei]

@Niels-Be Would you be open on writing benchmarks tests for the affected code to measure the performance impact?