Fixed lldb 'settings set term-widt 70' command not working.

rdar://problem/8449849

llvm-svn: 114328
This commit is contained in:
Johnny Chen 2010-09-20 16:36:43 +00:00
parent 201a88591d
commit ea9fc18163
2 changed files with 1 additions and 4 deletions

View File

@ -1311,7 +1311,7 @@ DebuggerInstanceSettings::ValidTermWidthValue (const char *value, Error err)
char *end = NULL;
const uint32_t width = ::strtoul (value, &end, 0);
if (end && end == '\0')
if (end && end[0] == '\0')
{
if (width >= 10 || width <= 1024)
valid = true;

View File

@ -30,9 +30,6 @@ class SettingsCommandTestCase(TestBase):
self.expect("settings show",
substrs = ["prompt (string) = 'lldb2'"])
@unittest2.expectedFailure
# rdar://problem/8449849
# settings test case failure (probably after r114252)
def test_set_term_width(self):
"""Test that 'set term-width' actually changes the term-width."""