Skip to content
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.

Sign up for GitHub

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

[DO NOT MERGE] Test reviewdog #53

Closed
wants to merge 4 commits into from
Closed

[DO NOT MERGE] Test reviewdog #53

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
de02eac
Update identity.hpp
wusatosi Oct 19, 2024
7aeea8a
Adds trailing white spaces
wusatosi Oct 19, 2024
a0a6ba0
trigger CI
wusatosi Oct 21, 2024
3020480
Merge branch 'main' into bad-format
wusatosi Nov 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .github/CODEOWNERS
View file
Open in desktop
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
# Codeowners for reviews on PRs

* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey
* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey

Comment on lines +4 to +5
Copy link
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey
* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey

10 changes: 9 additions & 1 deletion include/beman/exemplar/identity.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,15 @@ struct __is_transparent; // not defined
struct identity {
// Returns `t`.
template <class T>
constexpr T&& operator()(T&& t) const noexcept {
#if defined(__cpp_constexpr)
constexpr
#endif
T&&
operator()(T&& t) const noexcept {
int
size
=
0;
Comment on lines +38 to +41
Copy link
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
int
size
=
0;
int size = 0;

return std::forward<T>(t);
}

Expand Down
Loading