While examining this class for possible use in lldb, I noticed two
- it spits out parsing errors directly to stderr
- the loclists parser can incorrectly return valid location lists when parsing malformed (truncated) data
I improve the stderr situation by making the parseOneLocationList
functions return Expected<T>s. The errors are still dumped to stderr by
their callers, so this is only a partial fix, but it is enough for my
use case, as I intend to parse the locations lists one by one.
I fix the behavior in the truncated scenario by making the parser
explictly check for success. This makes the code a bit clunky, as the
DataExtractor API makes it really hard to do that, but this is the best
implementation I came up with given the current API.
I also add tests for handling the error cases, as they currently have no