Fix ZStream.fromInputStream

[iravid]

cc @reibitto

[reibitto]

I wonder if it would make sense to have a naming convention to serve the same purpose as this warning. Chunk.fromArray could copy by default and then the alternate "unsafe" one would be the fast version. Depends if there is already a precedent for that in the ZIO codebase.

Also when seeing this, I also thought about if there might be a use for a Chunk.fromArray[A](array: Array[A], start: Int, len: Int) version.

[iravid]

@reibitto I like the slicing version! Could be a nice PR if you have the cycles.

Re the naming: I think a warning is sufficient because we're not going to add the other variant. The caller can always defensively copy if required.

[simpadjo]

Maybe I'm too late for the party, but here is my take:

ZStream.managed(ZManaged.effectTotal(is)).flatMap(capturedIs => {
      ZStream.fromEffect(
        blocking.effectBlocking({
          val buf = Array.ofDim[Byte](chunkSize)
          val res = capturedIs.read(buf)
          if(res < 0) {
            Chunk.empty
          } else{
            val arr = if(res < chunkSize){
              buf.take(res)
            } else buf
            Chunk.fromArray(arr)
          }
        }).refineToOrDie[IOException]).takeWhile(_.nonEmpty).flattenChunks
    })