Skip to content

Add benchmark tests to path/posix #10691

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

Merged
merged 1 commit into from
Nov 28, 2013
Merged

Add benchmark tests to path/posix #10691

merged 1 commit into from
Nov 28, 2013

Conversation

g3xzh
Copy link
Contributor

@g3xzh g3xzh commented Nov 27, 2013

I have written some benchmark tests to push, push_many, join,
join_many and ends_with_path.

Let me know what you think (@cmr).
Thanks in advance.

I have written some benchmark tests to `push`, `push_many`, `join`,
`join_many` and `ends_with_path`.
@emberian
Copy link
Member

These look good, but I'm a bit worried that LLVM will optimize these right out. When you run the benchmarks, does it show them as taking a non-zero time? (You can do this with rustc --cfg stage2 --test bench.rs; ./std --bench path)

bors added a commit that referenced this pull request Nov 28, 2013
I have written some benchmark tests to `push`, `push_many`, `join`,
`join_many` and `ends_with_path`.

Let me know what you think (@cmr).
Thanks in advance.
@bors bors closed this Nov 28, 2013
@bors bors merged commit 26ba64d into rust-lang:master Nov 28, 2013
@g3xzh g3xzh deleted the benchm branch December 25, 2013 00:30
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 20, 2023
…archo

fix: warn on empty line outer AttrKind::DocComment

changelog: [`empty_line_after_doc_comments`]: add lint for checking empty lines after rustdoc comments.

Fixes: rust-lang#10395
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.

3 participants