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.

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

[CP #1591 > support/v5.13] [Mon / Qt Apps] Fixed filter-function #1594

Merged
merged 1 commit into from
May 15, 2024

Conversation

eclipse-ecal-bot
Copy link
Contributor

Cherry-pick

Cherry-picked PR #1591 to branch support/v5.13.
The cherry-pick was successful.

Please review the changes and rebase-merge if desired.

The MultiColumnSortFilterProxyModel now supports QRegexp and QRegularExpression depending on the Qt Version
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

if (!filter_regexp.isEmpty())
{
// Use QRegExp
for (const int column : filter_columns_)
Copy link
Contributor

Choose a reason for hiding this comment

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

warning: replace loop by 'std::any_of()' [readability-use-anyofallof]

    for (const int column : filter_columns_)
    ^

// Use QRegularExpression, as QRegExp is empty
QRegularExpression const filter_regularexpression = filterRegularExpression();

for (const int column : filter_columns_)
Copy link
Contributor

Choose a reason for hiding this comment

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

warning: replace loop by 'std::any_of()' [readability-use-anyofallof]

    for (const int column : filter_columns_)
    ^

@FlorianReimold FlorianReimold added the Auto Approve Request Automatic PR Approval label May 15, 2024
@FlorianReimold FlorianReimold merged commit 2f6c78a into support/v5.13 May 15, 2024
11 of 12 checks passed
@KerstinKeller KerstinKeller deleted the cherry-pick/3e99f97/support/v5.13 branch October 2, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants