Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Support streaming and raw i/o in contribution #10

Merged
merged 1 commit into from
Jun 28, 2020
Merged

Conversation

porcuquine
Copy link

This replaces #8 and includes support for raw i/o.

This branch is currently used by filecoin-project/rust-fil-proofs#1188. Once this merges, and the changes are released, that PR can be finalized.

@porcuquine porcuquine changed the title Feat/stream raw Support streaming and raw i/o in contribution Jun 26, 2020
@porcuquine porcuquine mentioned this pull request Jun 26, 2020
@porcuquine porcuquine marked this pull request as ready for review June 26, 2020 21:51
@porcuquine porcuquine force-pushed the feat/stream-raw branch 3 times, most recently from b3fba3f to 0cd487f Compare June 27, 2020 00:42
}

this_chunk_size = usize::min(chunk_size, chunks_to_read - chunks_read);
h_chunk.truncate(0);

Choose a reason for hiding this comment

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

These are good uses of truncate.

}
}

fn dump_g1<W: Write>(mut writer: W, g1: &G1Affine, raw: bool) -> io::Result<usize> {

Choose a reason for hiding this comment

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

Is this not inline-able?

Copy link
Author

Choose a reason for hiding this comment

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

Pushed again with that.

@porcuquine porcuquine merged commit 96c979d into master Jun 28, 2020
@porcuquine porcuquine deleted the feat/stream-raw branch August 13, 2020 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants