From af92957fd9831b43dd8768e39d0ba3dd34e820dc Mon Sep 17 00:00:00 2001 From: Rene Gollent Date: Fri, 24 Jul 2015 23:20:04 -0400 Subject: [PATCH] Debugger: Minor tweak to enumeration editor. - Don't return result of SelectOptionFor(). It's possible to attempt to edit an enum value that hasn't yet been initialized, in which case its current value might not map to any of the defined enumerations, and the resulting error would prevent editing from being allowed. --- .../user_interface/gui/value/TableCellEnumerationEditor.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/apps/debugger/user_interface/gui/value/TableCellEnumerationEditor.cpp b/src/apps/debugger/user_interface/gui/value/TableCellEnumerationEditor.cpp index 0ab8ef5af8..a164c1250f 100644 --- a/src/apps/debugger/user_interface/gui/value/TableCellEnumerationEditor.cpp +++ b/src/apps/debugger/user_interface/gui/value/TableCellEnumerationEditor.cpp @@ -44,7 +44,9 @@ TableCellEnumerationEditor::ConfigureOptions() if (!initialValue->ToVariant(integerValue)) return B_NO_MEMORY; - return SelectOptionFor(integerValue.ToInt32()); + SelectOptionFor(integerValue.ToInt32()); + + return B_OK; }