JSON::ResumableParser: error column does not match the input position

[mame]

The column in a ResumableParser error is counted from the start of the current internal buffer, which moves as input is consumed (and shrunk), so it does not correspond to the input.

require "json"
parser = JSON::ResumableParser.new
parser << "[" + "1," * 500 + "999"   # > 512 bytes, still open
parser.parse
parser << "@"
parser.parse
# expected: expected ',' or ']' after array value at line 1 column 1005
# actual:   expected ',' or ']' after array value at line 1 column 4

Should the position be tracked across shrinks? Personally I'm not sure a position within a stream is very helpful, so I wonder if it would be fine to just drop it.

[byroot]

Hum, that's a good catch, I didn't think of that.

I'm not sure a position within a stream is very helpful, so I wonder if it would be fine to just drop it.

Agreed. It's very useful to locate syntax errors in files, but in a stream it's a bit pointless.