caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove some lldbassert's from the packet checking code.

Browse Source 
Greg says he doesn't need these asserts anymore and since they cause occasional test suite
crashes, out they go.

llvm-svn: 268827
This commit is contained in:
Jim Ingham
2016-05-07 00:52:18 +00:00
parent 52bef53f86
commit 8bbfdcd181
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lldb/source/Utility/StringExtractorGDBRemote.cpp
View File

@@ -14,7 +14,6 @@
// Other libraries and framework includes
// Project includes
#include "Utility/StringExtractorGDBRemote.h"
#include "lldb/Utility/LLDBAssert.h"
StringExtractorGDBRemote::ResponseType
StringExtractorGDBRemote::GetResponseType () const
@@ -405,7 +404,6 @@ OKErrorNotSupportedResponseValidator(void *, const StringExtractorGDBRemote &res
case StringExtractorGDBRemote::eResponse:
break;
}
lldbassert(!"Packet validatation failed, check why this is happening");
return false;
}
@@ -438,7 +436,6 @@ JSONResponseValidator(void *, const StringExtractorGDBRemote &response)
}
break;
}
lldbassert(!"Packet validatation failed, check why this is happening");
return false;
}
@@ -463,7 +460,6 @@ ASCIIHexBytesResponseValidator(void *, const StringExtractorGDBRemote &response)
{
if (!isxdigit(ch))
{
lldbassert(!"Packet validatation failed, check why this is happening");
return false;
}
if (++valid_count >= 16)
@@ -473,7 +469,6 @@ ASCIIHexBytesResponseValidator(void *, const StringExtractorGDBRemote &response)
}
break;
}
lldbassert(!"Packet validatation failed, check why this is happening");
return false;
}