This is an archive of the discontinued Mercurial Phabricator instance.

clib: clean up include guards in the C/C++ header files
ClosedPublic

Authored by simpkins on Aug 24 2017, 4:39 PM.
Tags
None
Subscribers
None

Details

Reviewers
quark
Group Reviewers
Restricted Project
Commits
rFBHGX959ee56f6b6f: clib: clean up include guards in the C/C++ header files
Summary

Clean up the include guards to be more consistent and unique.

Some files used include guards like "KEY_H" and "STORE_H" which were not very
unique, and are more likely to collide with definitions provided by header
files from other projects. Some of the py-*.h files were missing include
guards altogether.

This corresponds to Facebook diff D5588670.

Test Plan

Confirmed the code still builds and passes tests.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

simpkins created this revision.Aug 24 2017, 4:39 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 24 2017, 4:39 PM
quark accepted this revision.Aug 24 2017, 5:48 PM
This revision is now accepted and ready to land.Aug 24 2017, 5:48 PM
This revision was automatically updated to reflect the committed changes.