jungm commented on PR #145: URL: https://github.com/apache/johnzon/pull/145#issuecomment-5429552610
yeah you're right, this was all a huge hack because i thought currentEvent must point to the first real event (index 0 instead of -1), hence all of that advancing and masking it code. the tck even tests the other way around https://github.com/jakartaee/jsonp-api/blob/2.1.3-RELEASE/tck/tck-tests/src/main/java/ee/jakarta/tck/jsonp/api/jsonparsertests/ClientTests.java#L1862 So agree, we can probably just drop RewindableJsonParser entirely since its only delegating? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
