status-im/react-native
2
0
Fork 0
mirror of https://github.com/status-im/react-native.git synced 2025-02-23 14:48:25 +00:00
Code Issues Projects Releases Wiki Activity

Pass enums by value

Browse Source 
Summary:
@public

passes all enum values by value, not by reference.

Reviewed By: astreet

Differential Revision: D13156390

fbshipit-source-id: 56aea66c16ab3325594f67b9017afa18a678d281
This commit is contained in:
David Aurelio 2018-11-22 08:06:25 -08:00 committed by Facebook Github Bot
parent 64d162e7c6
commit c34ad17c94
3 changed files with 33 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
ReactCommon/yoga/yoga/YGNode.cpp
View File

@ -12,7 +12,7 @@
using namespace facebook; using namespace facebook;
YGFloatOptional YGNode::getLeadingPosition( YGFloatOptional YGNode::getLeadingPosition(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float axisSize) const { const float axisSize) const {
if (YGFlexDirectionIsRow(axis)) { if (YGFlexDirectionIsRow(axis)) {
const YGValue* leadingPosition = const YGValue* leadingPosition =
@ -31,7 +31,7 @@ YGFloatOptional YGNode::getLeadingPosition(
} }
YGFloatOptional YGNode::getTrailingPosition( YGFloatOptional YGNode::getTrailingPosition(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float axisSize) const { const float axisSize) const {
if (YGFlexDirectionIsRow(axis)) { if (YGFlexDirectionIsRow(axis)) {
const YGValue* trailingPosition = const YGValue* trailingPosition =
@ -49,7 +49,7 @@ YGFloatOptional YGNode::getTrailingPosition(
: YGResolveValue(*trailingPosition, axisSize); : YGResolveValue(*trailingPosition, axisSize);
} }
bool YGNode::isLeadingPositionDefined(const YGFlexDirection& axis) const { bool YGNode::isLeadingPositionDefined(const YGFlexDirection axis) const {
return (YGFlexDirectionIsRow(axis) && return (YGFlexDirectionIsRow(axis) &&
YGComputedEdgeValue(style_.position, YGEdgeStart, &YGValueUndefined) YGComputedEdgeValue(style_.position, YGEdgeStart, &YGValueUndefined)
->unit != YGUnitUndefined) || ->unit != YGUnitUndefined) ||
@ -57,7 +57,7 @@ bool YGNode::isLeadingPositionDefined(const YGFlexDirection& axis) const {
->unit != YGUnitUndefined; ->unit != YGUnitUndefined;
} }
bool YGNode::isTrailingPosDefined(const YGFlexDirection& axis) const { bool YGNode::isTrailingPosDefined(const YGFlexDirection axis) const {
return (YGFlexDirectionIsRow(axis) && return (YGFlexDirectionIsRow(axis) &&
YGComputedEdgeValue(style_.position, YGEdgeEnd, &YGValueUndefined) YGComputedEdgeValue(style_.position, YGEdgeEnd, &YGValueUndefined)
->unit != YGUnitUndefined) || ->unit != YGUnitUndefined) ||
@ -66,7 +66,7 @@ bool YGNode::isTrailingPosDefined(const YGFlexDirection& axis) const {
} }
YGFloatOptional YGNode::getLeadingMargin( YGFloatOptional YGNode::getLeadingMargin(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
if (YGFlexDirectionIsRow(axis) && if (YGFlexDirectionIsRow(axis) &&
style_.margin[YGEdgeStart].unit != YGUnitUndefined) { style_.margin[YGEdgeStart].unit != YGUnitUndefined) {
@ -79,7 +79,7 @@ YGFloatOptional YGNode::getLeadingMargin(
} }
YGFloatOptional YGNode::getTrailingMargin( YGFloatOptional YGNode::getTrailingMargin(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
if (YGFlexDirectionIsRow(axis) && if (YGFlexDirectionIsRow(axis) &&
style_.margin[YGEdgeEnd].unit != YGUnitUndefined) { style_.margin[YGEdgeEnd].unit != YGUnitUndefined) {
@ -92,7 +92,7 @@ YGFloatOptional YGNode::getTrailingMargin(
} }
YGFloatOptional YGNode::getMarginForAxis( YGFloatOptional YGNode::getMarginForAxis(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
return getLeadingMargin(axis, widthSize) + getTrailingMargin(axis, widthSize); return getLeadingMargin(axis, widthSize) + getTrailingMargin(axis, widthSize);
} }
@ -204,7 +204,7 @@ void YGNode::setLayoutDimension(float dimension, int index) {
// If both left and right are defined, then use left. Otherwise return // If both left and right are defined, then use left. Otherwise return
// +left or -right depending on which is defined. // +left or -right depending on which is defined.
YGFloatOptional YGNode::relativePosition( YGFloatOptional YGNode::relativePosition(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float axisSize) const { const float axisSize) const {
if (isLeadingPositionDefined(axis)) { if (isLeadingPositionDefined(axis)) {
return getLeadingPosition(axis, axisSize); return getLeadingPosition(axis, axisSize);
@ -424,7 +424,7 @@ bool YGNode::isNodeFlexible() {
(resolveFlexGrow() != 0 || resolveFlexShrink() != 0)); (resolveFlexGrow() != 0 || resolveFlexShrink() != 0));
} }
float YGNode::getLeadingBorder(const YGFlexDirection& axis) const { float YGNode::getLeadingBorder(const YGFlexDirection axis) const {
if (YGFlexDirectionIsRow(axis) && if (YGFlexDirectionIsRow(axis) &&
style_.border[YGEdgeStart].unit != YGUnitUndefined && style_.border[YGEdgeStart].unit != YGUnitUndefined &&
!yoga::isUndefined(style_.border[YGEdgeStart].value) && !yoga::isUndefined(style_.border[YGEdgeStart].value) &&
@ -437,7 +437,7 @@ float YGNode::getLeadingBorder(const YGFlexDirection& axis) const {
return YGFloatMax(computedEdgeValue, 0.0f); return YGFloatMax(computedEdgeValue, 0.0f);
} }
float YGNode::getTrailingBorder(const YGFlexDirection& flexDirection) const { float YGNode::getTrailingBorder(const YGFlexDirection flexDirection) const {
if (YGFlexDirectionIsRow(flexDirection) && if (YGFlexDirectionIsRow(flexDirection) &&
style_.border[YGEdgeEnd].unit != YGUnitUndefined && style_.border[YGEdgeEnd].unit != YGUnitUndefined &&
!yoga::isUndefined(style_.border[YGEdgeEnd].value) && !yoga::isUndefined(style_.border[YGEdgeEnd].value) &&
@ -452,7 +452,7 @@ float YGNode::getTrailingBorder(const YGFlexDirection& flexDirection) const {
} }
YGFloatOptional YGNode::getLeadingPadding( YGFloatOptional YGNode::getLeadingPadding(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
const YGFloatOptional& paddingEdgeStart = const YGFloatOptional& paddingEdgeStart =
YGResolveValue(style_.padding[YGEdgeStart], widthSize); YGResolveValue(style_.padding[YGEdgeStart], widthSize);
@ -469,7 +469,7 @@ YGFloatOptional YGNode::getLeadingPadding(
} }
YGFloatOptional YGNode::getTrailingPadding( YGFloatOptional YGNode::getTrailingPadding(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
if (YGFlexDirectionIsRow(axis) && if (YGFlexDirectionIsRow(axis) &&
style_.padding[YGEdgeEnd].unit != YGUnitUndefined && style_.padding[YGEdgeEnd].unit != YGUnitUndefined &&
@ -486,14 +486,14 @@ YGFloatOptional YGNode::getTrailingPadding(
} }
YGFloatOptional YGNode::getLeadingPaddingAndBorder( YGFloatOptional YGNode::getLeadingPaddingAndBorder(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
return getLeadingPadding(axis, widthSize) + return getLeadingPadding(axis, widthSize) +
YGFloatOptional(getLeadingBorder(axis)); YGFloatOptional(getLeadingBorder(axis));
} }
YGFloatOptional YGNode::getTrailingPaddingAndBorder( YGFloatOptional YGNode::getTrailingPaddingAndBorder(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const { const float widthSize) const {
return getTrailingPadding(axis, widthSize) + return getTrailingPadding(axis, widthSize) +
YGFloatOptional(getTrailingBorder(axis)); YGFloatOptional(getTrailingBorder(axis));

28
ReactCommon/yoga/yoga/YGNode.h
View File

@ -33,7 +33,7 @@ struct YGNode {
{YGValueUndefined, YGValueUndefined}}; {YGValueUndefined, YGValueUndefined}};
YGFloatOptional relativePosition( YGFloatOptional relativePosition(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float axisSize) const; const float axisSize) const;
public: public:
@ -138,35 +138,35 @@ struct YGNode {
// Methods related to positions, margin, padding and border // Methods related to positions, margin, padding and border
YGFloatOptional getLeadingPosition( YGFloatOptional getLeadingPosition(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float axisSize) const; const float axisSize) const;
bool isLeadingPositionDefined(const YGFlexDirection& axis) const; bool isLeadingPositionDefined(const YGFlexDirection axis) const;
bool isTrailingPosDefined(const YGFlexDirection& axis) const; bool isTrailingPosDefined(const YGFlexDirection axis) const;
YGFloatOptional getTrailingPosition( YGFloatOptional getTrailingPosition(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float axisSize) const; const float axisSize) const;
YGFloatOptional getLeadingMargin( YGFloatOptional getLeadingMargin(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
YGFloatOptional getTrailingMargin( YGFloatOptional getTrailingMargin(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
float getLeadingBorder(const YGFlexDirection& flexDirection) const; float getLeadingBorder(const YGFlexDirection flexDirection) const;
float getTrailingBorder(const YGFlexDirection& flexDirection) const; float getTrailingBorder(const YGFlexDirection flexDirection) const;
YGFloatOptional getLeadingPadding( YGFloatOptional getLeadingPadding(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
YGFloatOptional getTrailingPadding( YGFloatOptional getTrailingPadding(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
YGFloatOptional getLeadingPaddingAndBorder( YGFloatOptional getLeadingPaddingAndBorder(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
YGFloatOptional getTrailingPaddingAndBorder( YGFloatOptional getTrailingPaddingAndBorder(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
YGFloatOptional getMarginForAxis( YGFloatOptional getMarginForAxis(
const YGFlexDirection& axis, const YGFlexDirection axis,
const float widthSize) const; const float widthSize) const;
// Setters // Setters

10
ReactCommon/yoga/yoga/Yoga.cpp
View File

@ -1220,7 +1220,7 @@ static inline bool YGNodeIsLayoutDimDefined(
static YGFloatOptional YGNodeBoundAxisWithinMinAndMax( static YGFloatOptional YGNodeBoundAxisWithinMinAndMax(
const YGNodeRef node, const YGNodeRef node,
const YGFlexDirection& axis, const YGFlexDirection axis,
const float value, const float value,
const float axisSize) { const float axisSize) {
YGFloatOptional min; YGFloatOptional min;
@ -2423,10 +2423,10 @@ static void YGJustifyMainAxis(
const YGNodeRef node, const YGNodeRef node,
YGCollectFlexItemsRowValues& collectedFlexItemsValues, YGCollectFlexItemsRowValues& collectedFlexItemsValues,
const uint32_t startOfLineIndex, const uint32_t startOfLineIndex,
const YGFlexDirection& mainAxis, const YGFlexDirection mainAxis,
const YGFlexDirection& crossAxis, const YGFlexDirection crossAxis,
const YGMeasureMode& measureModeMainDim, const YGMeasureMode measureModeMainDim,
const YGMeasureMode& measureModeCrossDim, const YGMeasureMode measureModeCrossDim,
const float mainAxisownerSize, const float mainAxisownerSize,
const float ownerWidth, const float ownerWidth,
const float availableInnerMainDim, const float availableInnerMainDim,