New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[summarize] Implement diff
support for comparing profiles
#50
Conversation
|
||
fn write_results_json(file: &PathBuf, results: impl Serialize) -> Result<(), Box<dyn Error>> { | ||
let file = BufWriter::new(File::create(file.with_extension("json"))?); | ||
serde_json::to_writer(file, &results)?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would do the same here -- instead of a BufWriter just write to a vec in memory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think for now, I'd prefer to leave the standard BufReader
`BufWriter` alone. These tools run pretty quick already so I don't think this is going to make a big difference.
I think this is ready to merge. @Mark-Simulacrum is that ok with you? |
Feel free to merge when ready -- I forget if we have bors or anything setup here so won't do it myself in this case |
Closes #33