Skip to content

Commit

Permalink
Debugger: Minor tweak to enumeration editor.
Browse files Browse the repository at this point in the history
- 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.
  • Loading branch information
anevilyak committed Jul 25, 2015
1 parent 410b38b commit af92957
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,9 @@ TableCellEnumerationEditor::ConfigureOptions()
if (!initialValue->ToVariant(integerValue))
return B_NO_MEMORY;

return SelectOptionFor(integerValue.ToInt32());
SelectOptionFor(integerValue.ToInt32());

return B_OK;
}


Expand Down

0 comments on commit af92957

Please sign in to comment.