Update to build on Xcode 6.0: fixes for brackets logical operations and signed/unsigned issue
parent
3929725fe7
commit
ff5c13493e
|
|
@ -381,7 +381,7 @@ bool LLTimeCtrl::isHoursStringValid(const std::string& str)
|
|||
bool LLTimeCtrl::isMinutesStringValid(const std::string& str)
|
||||
{
|
||||
U32 minutes;
|
||||
if (!LLStringUtil::convertToU32(str, minutes) || (minutes <= MINUTES_MAX) && str.length() < 3)
|
||||
if (!LLStringUtil::convertToU32(str, minutes) || ((minutes <= MINUTES_MAX) && str.length() < 3))
|
||||
return true;
|
||||
|
||||
return false;
|
||||
|
|
@ -415,7 +415,8 @@ U32 LLTimeCtrl::parseHours(const std::string& str)
|
|||
U32 LLTimeCtrl::parseMinutes(const std::string& str)
|
||||
{
|
||||
U32 minutes;
|
||||
if (LLStringUtil::convertToU32(str, minutes) && (minutes >= MINUTES_MIN) && (minutes <= MINUTES_MAX))
|
||||
// not sure of this fix - clang doesnt like compare minutes U32 to >= MINUTES_MIN (0) but MINUTES_MIN can change
|
||||
if (LLStringUtil::convertToU32(str, minutes) && ((S32)minutes >= MINUTES_MIN) && ((S32)minutes <= MINUTES_MAX))
|
||||
{
|
||||
return minutes;
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue