Thanks. To be reviewed.

Q: do we have benchmarking for the iterator? Seems like a lot of people will interact with JSON via it.

We do have some benchmarks. Look in the benchmark directory. We have not done work to optimize it... I suspect that switching to a flat 16-byte tape might help.

@DBJDBJ Would you have a look?

@lemire OK

I am looking at ~/simdjson/parsedjson.h and ~/simdjson/parsedjsoniterator.h, right?

The PR diff is at https://github.com/lemire/simdjson/pull/421/files

The issue at #420

"...switching to a flat 16-byte tape..."

@lemire, tape is a member of ParsedJson,

class ParsedJson {
. . .
std::unique_ptr<uint64_t[]> tape;
. . .
} ;

I assume you are asking me to look into improving ParsedJson and ParsedJson::BasicIterator ? Thus, I have asked about ~/simdjson/parsedjson.h and ~/simdjson/parsedjsoniterator.h ...

On the fix for #420 and #421, I can only guess, which somehow I would not like to do :) (ps: I would like to ask the requestor if he has proven the fix by running the tests . I am not clear who has found and proven the issue.)