Page MenuHomePhabricator

denzor200 (Denis Mikhailov)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 3 2022, 10:46 AM (20 w, 4 d)

Recent Activity

Feb 26 2022

denzor200 planned changes to D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.
Feb 26 2022, 4:14 AM · Restricted Project

Feb 25 2022

denzor200 updated the diff for D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.
Feb 25 2022, 5:03 PM · Restricted Project
denzor200 added a comment to D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.

My apologize for the delay! Diff was updated, implemented detection of known failure cases in the matcher and not issue a fixit, just only diagnostic.

Feb 25 2022, 4:11 PM · Restricted Project
denzor200 updated the diff for D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.
Feb 25 2022, 4:06 PM · Restricted Project

Jan 9 2022

denzor200 added inline comments to D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.
Jan 9 2022, 11:36 PM · Restricted Project
denzor200 updated the diff for D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.

clang-format

Jan 9 2022, 11:20 PM · Restricted Project
denzor200 updated the diff for D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.

review

Jan 9 2022, 11:17 PM · Restricted Project
denzor200 added a comment to D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.

I opened a similar issue for converting Qt's foreach to a range for loop.

However this check lands, it should be a simple generalization to have it process Qt foreach loops as well, so perhaps a check name like modernize-foreach-to-range-for would be better?

Jan 9 2022, 10:20 PM · Restricted Project

Jan 3 2022

denzor200 requested review of D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check.
Jan 3 2022, 11:28 PM · Restricted Project