This is an archive of the discontinued LLVM Phabricator instance.

[scudo] s/unsigned long/size_t/ for __scudo_set_rss_limit
ClosedPublic

Authored by cryptoad on Jan 3 2018, 10:21 AM.

Details

Summary

__scudo_set_rss_limit's LimitMb should really be a size_t. Update
accordingly the prototype. To avoid the NOLINT and conform with the other
Sanitizers, use the sanitizers types for the internal definition. This should
have no functional change.

Additionally, capitalize a variable name to follow the LLVM coding standards.

Event Timeline

cryptoad created this revision.Jan 3 2018, 10:21 AM
Herald added a subscriber: Restricted Project. · View Herald TranscriptJan 3 2018, 10:21 AM
alekseyshl accepted this revision.Jan 3 2018, 8:27 PM
This revision is now accepted and ready to land.Jan 3 2018, 8:27 PM
This revision was automatically updated to reflect the committed changes.