stream: minor fix for ReadableState

[cynron]

1. remove unused flag ranOut from ReadableState, ranOut had been unused since 0f8de5e

2. fix the comments for flag sync of ReadableState.

Checklist

• make -j4 test (UNIX) passes
• commit message follows commit guidelines

Affected core subsystem(s)

• stream

[addaleax]

CI: https://ci.nodejs.org/job/node-test-commit/7647/

[jasnell]

@nodejs/streams

[jasnell]

Given that this removes a property, this is potentially semver-major. It shouldn't be, but we need to be certain.

[mcollina]

CITGM: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/563/

[mcollina]

I think we are going to release this as semver-minor in readable-stream. If that does not cause issues, we can release this as semver-minor also in Node.

ReadableState is undocumented atm.

[mcollina]

Landed as 1004b9b and 202b07f

[addaleax]

@mcollina Just ICYMI, our current policy is that at least 2 CTC members need to sign off on semver-major changes before landing them. I don’t think that’s a problem here because it’s not something that’s really covered by our API (as you pointed out)… I just added the label because, at least from Node’s perspective, I don’t think there’s a reason to not consider this semver-major.

[mcollina]

Oh, I thought this was sitting there for a while, and just reduced the pull requests count by one.
Thanks for noting.