interface & app_server: Fix missing parentheses near ternary operators.
+ has greater operator precendence than ?, so we need parentheses when using both in expressions. Both of these look like actual bugs. Spotted by Clang.
This commit is contained in:
@@ -352,5 +352,5 @@ _BMCMenuBar_::_Init()
|
||||
left = right = be_control_look->DefaultLabelSpacing();
|
||||
|
||||
SetItemMargins(left, top,
|
||||
right + fShowPopUpMarker ? kPopUpIndicatorWidth : 0, bottom);
|
||||
right + (fShowPopUpMarker ? kPopUpIndicatorWidth : 0), bottom);
|
||||
}
|
||||
|
||||
@@ -299,7 +299,7 @@ DelayedMessageData::DelayedMessageData(int32 code, bigtime_t delay,
|
||||
bool isSpecificTime)
|
||||
:
|
||||
fCode(code),
|
||||
fScheduledTime(delay + isSpecificTime ? 0 : system_time()),
|
||||
fScheduledTime(delay + (isSpecificTime ? 0 : system_time())),
|
||||
fValid(false),
|
||||
|
||||
fAttachments(3, true),
|
||||
|
||||
Reference in New Issue
Block a user