Skip to content

Fix clang tidy warnings #105

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

Jump to bottom
Merged
dietmarkuehl merged 23 commits into from
Dec 22, 2024
Merged

Fix clang tidy warnings #105

dietmarkuehl merged 23 commits into from
Dec 22, 2024

Conversation

@dietmarkuehl
Copy link
Member

@dietmarkuehl dietmarkuehl commented Dec 21, 2024

No description provided.

dietmarkuehl and others added 23 commits December 21, 2024 22:08
@dietmarkuehl
address another helping of clang-tidy reports
4c701ed
@dietmarkuehl
remove commented, unnecessary code
734df0c
@dietmarkuehl
fixed the noexception specification for connect and connect_all
cd50ced
@dietmarkuehl
implemented noexcept for basic_operation/basic_sender
059d7df
@dietmarkuehl
fixed a number of gcc warnings (and hopefully an error)
ab3a204
@dietmarkuehl
another attempt at fixing a gcc compilation issue
a38953d
@dietmarkuehl
fixed more compiler warnings
0ba823e
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
c396779 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
819eb92 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
349f4ab 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-split.test.cpp
04c570e 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
635e2a5 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
04ffd54 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
0d5b969 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
11f602a 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
f819a49 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
ed1a53a 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
4302192 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
951c150 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
b6ffa26 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
2547dce 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
a230fa5 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl @github-actions
Update tests/beman/execution26/exec-snd-expos.test.cpp
ce24d6a 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@dietmarkuehl dietmarkuehl marked this pull request as ready for review December 22, 2024 01:00
@dietmarkuehl dietmarkuehl merged commit 5b44c98 into main Dec 22, 2024
12 checks passed
@dietmarkuehl dietmarkuehl deleted the fix-clang-tidy-warnings branch December 22, 2024 01:10
dietmarkuehl added a commit that referenced this pull request Jan 21, 2025
@dietmarkuehl
Fix clang tidy warnings (#105)
ee2bdae 
- fixed various clang-tidy warnings
- fixed various gcc warnings
- implemented missing noexcept specifications in some places to fix clang-tidy warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@camio camio Awaiting requested review from camio camio is a code owner

@neatudarius neatudarius Awaiting requested review from neatudarius neatudarius is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dietmarkuehl