Page MenuHomePhabricator

Modernize the rest of the Find.* API (NFC)

Authored by aprantl on Oct 17 2019, 10:31 AM.



This patch removes the size_t return value and the append parameter from the remainder of the Fund.* functions in LLDB's internal API. As in the previous patches, this is motivated by the fact that these parameters aren't really used, and in the case of the append parameter were frequently implemented incorrectly.

Diff Detail

Event Timeline

aprantl created this revision.Oct 17 2019, 10:31 AM
JDevlieghere accepted this revision.Oct 17 2019, 11:00 AM
This revision is now accepted and ready to land.Oct 17 2019, 11:00 AM
labath accepted this revision.Oct 17 2019, 11:12 AM



I'm pretty sure these Clears are not needed.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptOct 17 2019, 1:04 PM

This breaks the build due to lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp:1355:8: error: unused variable 'old_size' [-Werror,-Wunused-variable]

I will check in a fix shortly.

ted added a subscriber: ted.Jan 8 2020, 12:30 PM
This comment was removed by ted.