DaddyFrosty/yoga
1
0
Fork 0
Code Issues 96 Pull Requests 31 Actions Packages Projects Releases 33 Wiki Activity

Use switch instead of ifelse chain

Browse Source 
Summary: easier to read

Reviewed By: gkassabli

Differential Revision: D4036445

fbshipit-source-id: 2159946f53507ff3d7505795e5baa5f820cdb65c
This commit is contained in:
Emil Sjolander
2016-10-18 10:01:16 -07:00
committed by Facebook Github Bot
parent 9c3970dd75
commit d1d9326fa4
1 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
CSSLayout/CSSLayout.c
View File

@@ -1855,15 +1855,21 @@ static void layoutNodeImpl(const CSSNodeRef node,
float crossDimLead = 0;
float currentLead = leadingPaddingAndBorderCross;
const CSSAlign alignContent = node->style.alignContent;
if (alignContent == CSSAlignFlexEnd) {
currentLead += remainingAlignContentDim;
} else if (alignContent == CSSAlignCenter) {
currentLead += remainingAlignContentDim / 2;
} else if (alignContent == CSSAlignStretch) {
if (availableInnerCrossDim > totalLineCrossDim) {
crossDimLead = (remainingAlignContentDim / lineCount);
}
switch (node->style.alignContent) {
case CSSAlignFlexEnd:
currentLead += remainingAlignContentDim;
break;
case CSSAlignCenter:
currentLead += remainingAlignContentDim / 2;
break;
case CSSAlignStretch:
if (availableInnerCrossDim > totalLineCrossDim) {
crossDimLead = (remainingAlignContentDim / lineCount);
}
break;
case CSSAlignAuto:
case CSSAlignFlexStart:
break;
}
uint32_t endIndex = 0;