Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D149999

[clang-tidy] [test] Narrow down a special case to MSVC mode
ClosedPublic
Actions

Authored by mstorsjo on May 5 2023, 2:47 PM.

Details

Reviewers
alvinhochun
aaron.ballman
hans
Commits
rGb80febdf43d3: [clang-tidy] [test] Narrow down a special case to MSVC mode
Summary

For MinGW targets, size_t isn't a compiler defined type, just like
for unix targets.

Diff Detail

Event Timeline

mstorsjo created this revision.May 5 2023, 2:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 5 2023, 2:47 PM
Herald added subscribers: PiotrZSL, carlosgalvezp, xazax.hun. · View Herald Transcript
mstorsjo requested review of this revision.May 5 2023, 2:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 5 2023, 2:47 PM
Harbormaster completed remote builds in B230332: Diff 519980.May 5 2023, 3:03 PM
hans accepted this revision.May 8 2023, 7:38 AM

lgtm

This revision is now accepted and ready to land.May 8 2023, 7:38 AM
Closed by commit rGb80febdf43d3: [clang-tidy] [test] Narrow down a special case to MSVC mode (authored by mstorsjo). · Explain WhyMay 9 2023, 3:08 PM
This revision was automatically updated to reflect the committed changes.
mstorsjo added a commit: rGb80febdf43d3: [clang-tidy] [test] Narrow down a special case to MSVC mode.

Revision Contents
Changeset List

PathSize
clang-tools-extra/
test/
clang-tidy/
checkers/
readability/
2 lines

Diff 520835

View Options

clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-standard-types.h

Loading...