diff options
author | Greg Clayton <gclayton@apple.com> | 2013-01-25 18:06:21 +0000 |
---|---|---|
committer | Greg Clayton <gclayton@apple.com> | 2013-01-25 18:06:21 +0000 |
commit | c7bece56faa5eef1c3d141d0c0b0b68b28a9aed2 (patch) | |
tree | 9a0132fc3b0bb4f38d06a0f352ee75ac57994771 /lldb/source/Core/ValueObjectList.cpp | |
parent | d0ed6c249dbd6bd488b6491b536a387548c00f7e (diff) | |
download | bcm5719-llvm-c7bece56faa5eef1c3d141d0c0b0b68b28a9aed2.tar.gz bcm5719-llvm-c7bece56faa5eef1c3d141d0c0b0b68b28a9aed2.zip |
<rdar://problem/13069948>
Major fixed to allow reading files that are over 4GB. The main problems were that the DataExtractor was using 32 bit offsets as a data cursor, and since we mmap all of our object files we could run into cases where if we had a very large core file that was over 4GB, we were running into the 4GB boundary.
So I defined a new "lldb::offset_t" which should be used for all file offsets.
After making this change, I enabled warnings for data loss and for enexpected implicit conversions temporarily and found a ton of things that I fixed.
Any functions that take an index internally, should use "size_t" for any indexes and also should return "size_t" for any sizes of collections.
llvm-svn: 173463
Diffstat (limited to 'lldb/source/Core/ValueObjectList.cpp')
-rw-r--r-- | lldb/source/Core/ValueObjectList.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lldb/source/Core/ValueObjectList.cpp b/lldb/source/Core/ValueObjectList.cpp index 9f7185aa668..180d4a0eaf1 100644 --- a/lldb/source/Core/ValueObjectList.cpp +++ b/lldb/source/Core/ValueObjectList.cpp @@ -61,20 +61,20 @@ ValueObjectList::Append (const ValueObjectList &valobj_list) } -uint32_t +size_t ValueObjectList::GetSize() const { return m_value_objects.size(); } void -ValueObjectList::Resize (uint32_t size) +ValueObjectList::Resize (size_t size) { m_value_objects.resize (size); } lldb::ValueObjectSP -ValueObjectList::GetValueObjectAtIndex (uint32_t idx) +ValueObjectList::GetValueObjectAtIndex (size_t idx) { lldb::ValueObjectSP valobj_sp; if (idx < m_value_objects.size()) @@ -83,7 +83,7 @@ ValueObjectList::GetValueObjectAtIndex (uint32_t idx) } lldb::ValueObjectSP -ValueObjectList::RemoveValueObjectAtIndex (uint32_t idx) +ValueObjectList::RemoveValueObjectAtIndex (size_t idx) { lldb::ValueObjectSP valobj_sp; if (idx < m_value_objects.size()) @@ -95,7 +95,7 @@ ValueObjectList::RemoveValueObjectAtIndex (uint32_t idx) } void -ValueObjectList::SetValueObjectAtIndex (uint32_t idx, const ValueObjectSP &valobj_sp) +ValueObjectList::SetValueObjectAtIndex (size_t idx, const ValueObjectSP &valobj_sp) { if (idx >= m_value_objects.size()) m_value_objects.resize (idx + 1); |