Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@gnp
Copy link
Contributor

@gnp gnp commented Mar 10, 2023

Allows a user who has a ZStream that may or may not be gzipped to easily operate on the uncompressed version.

Copy link
Contributor

@adamgfraser adamgfraser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Left a coupe of minor comments.

}

private[stream] trait ZPipelinePlatformSpecificConstructors {
private val gzipMagic: zio.Chunk[Byte] = zio.Chunk(0x1f.toByte, 0x8b.toByte)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe just prefix?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could also inline this into the implementation since it is only used in one place.

private[stream] trait ZPipelinePlatformSpecificConstructors {
private val gzipMagic: zio.Chunk[Byte] = zio.Chunk(0x1f.toByte, 0x8b.toByte)

def autoGunzip[Env](
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gunzipAuto?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be great to add some documentation about what this does.

@gnp gnp requested a review from adamgfraser March 10, 2023 15:40
@adamgfraser adamgfraser merged commit 65cfa7c into zio:series/2.x Mar 10, 2023
@gnp gnp deleted the autoGunzip branch April 16, 2023 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants