wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1001708: Don't use ternary operator with class const statics r=jesup

Browse Source
This commit is contained in:
Garvan Keeley 2014-04-27 00:02:17 -04:00
parent df2ea577c8
commit 5442d73014
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
media/webrtc/signaling/src/common/YuvStamper.cpp
View File

@ -346,7 +346,13 @@ static unsigned char *DIGITS[] = {
bool YuvStamper::WriteBit(bool one)
{
// A bit is mapped to a mSymbolWidth x mSymbolHeight square of luma data points.
unsigned char value = one ? sYOn : sYOff;
// Don't use ternary op.: https://bugzilla.mozilla.org/show_bug.cgi?id=1001708
unsigned char value;
if (one)
value = sYOn;
else
value = sYOff;
for (uint32_t y = 0; y < mSymbolHeight; y++) {
for (uint32_t x = 0; x < mSymbolWidth; x++) {
*(pYData + (mCursor.x + x) + ((mCursor.y + y) * mStride)) = value;
@ -452,8 +458,11 @@ static unsigned char *DIGITS[] = {
void YuvStamper::WritePixel(unsigned char *data, uint32_t x, uint32_t y) {
unsigned char *ptr = &data[y * mStride + x];
*ptr = (*ptr > sLumaThreshold) ? sLumaMin : sLumaMax;
return;
// Don't use ternary op.: https://bugzilla.mozilla.org/show_bug.cgi?id=1001708
if (*ptr > sLumaThreshold)
*ptr = sLumaMin;
else
*ptr = sLumaMax;
}
} // Namespace mozilla.