Page MenuHomePhabricator

[include-cleaner] Make Symbol (and Macro) hashable.
ClosedPublic

Authored by sammccall on Thu, Nov 24, 3:31 AM.

Details

Summary

For now, we decided not to add operator< or handle other variants.
(If we do so in future we may want to extract a base class).

Diff Detail

Event Timeline

sammccall created this revision.Thu, Nov 24, 3:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Nov 24, 3:31 AM
sammccall requested review of this revision.Thu, Nov 24, 3:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Nov 24, 3:31 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
kadircet accepted this revision.Thu, Nov 24, 4:40 AM

thanks, lgtm!

clang-tools-extra/include-cleaner/include/clang-include-cleaner/Types.h
146

probably left over?

This revision is now accepted and ready to land.Thu, Nov 24, 4:40 AM
sammccall marked an inline comment as done.Thu, Nov 24, 6:15 AM
This revision was automatically updated to reflect the committed changes.