yutakahirano / fetch-with-streams

Fetch API integrated with Streams.
66 stars 8 forks source link

Put enough clarification about the difference between the associated body and the body attribute of Response/Request class #7

Closed tyoshino closed 9 years ago

tyoshino commented 9 years ago

It's confusing that these different things are given the same name.

tyoshino commented 9 years ago

Maybe you cannot for now since the associated body is used in texts that you're not replacing in the draft. Please take this as TODO.

annevk commented 9 years ago

On their own they each make perfect sense. I would hate to start calling the request/response's body something else. And I would hate to rename the member on request/response that exposes it.

tyoshino commented 9 years ago

On their own they each make perfect sense.

Agreed

I would hate to start calling the request/response's body something else.

Hmm, ok. The name is not exposed to the web but already widely used across the specs and descriptive/simple enough. Let's give good description not to confuse readers maybe with enough notes.

And I would hate to rename the member on request/response that exposes it.

Agreed