diff --git a/Examples/UIExplorer/DatePickerAndroidExample.js b/Examples/UIExplorer/DatePickerAndroidExample.js index e59517d91..355d8cbd2 100644 --- a/Examples/UIExplorer/DatePickerAndroidExample.js +++ b/Examples/UIExplorer/DatePickerAndroidExample.js @@ -46,7 +46,7 @@ var DatePickerAndroidExample = React.createClass({ async showPicker(stateKey, options) { try { var newState = {}; - const {action, year, month, day} = await DatePickerAndroid.open(options); + const {action, year, month, day} = await DatePickerAndroid.open(options); if (action === DatePickerAndroid.dismissedAction) { newState[stateKey + 'Text'] = 'dismissed'; } else { diff --git a/Examples/UIExplorer/ImageEditingExample.js b/Examples/UIExplorer/ImageEditingExample.js index 868bbd25f..8cd966f62 100644 --- a/Examples/UIExplorer/ImageEditingExample.js +++ b/Examples/UIExplorer/ImageEditingExample.js @@ -230,7 +230,7 @@ class ImageCropper extends React.Component { ); this._minimumZoomScale = Math.max( this.props.size.width / this._scaledImageSize.width, - this.props.size.height / this._scaledImageSize.height + this.props.size.height / this._scaledImageSize.height ); this._updateTransformData( this._contentOffset, diff --git a/Examples/UIExplorer/TextInputExample.android.js b/Examples/UIExplorer/TextInputExample.android.js index a189f4f9d..2c8b48d64 100644 --- a/Examples/UIExplorer/TextInputExample.android.js +++ b/Examples/UIExplorer/TextInputExample.android.js @@ -416,7 +416,7 @@ exports.examples = [ secureTextEntry={true} style={styles.singleLine} /> - -@interface OCMArg : NSObject +@interface OCMArg : NSObject // constraining arguments diff --git a/Examples/UIExplorer/UIExplorerUnitTests/OCMock/OCMConstraint.h b/Examples/UIExplorer/UIExplorerUnitTests/OCMock/OCMConstraint.h index 777966ab7..25aa8bf49 100644 --- a/Examples/UIExplorer/UIExplorerUnitTests/OCMock/OCMConstraint.h +++ b/Examples/UIExplorer/UIExplorerUnitTests/OCMock/OCMConstraint.h @@ -17,7 +17,7 @@ #import -@interface OCMConstraint : NSObject +@interface OCMConstraint : NSObject + (instancetype)constraint; - (BOOL)evaluate:(id)value; diff --git a/Examples/UIExplorer/UIExplorerUnitTests/RCTBridgeTests.m b/Examples/UIExplorer/UIExplorerUnitTests/RCTBridgeTests.m index 337a223fd..9f2570bcd 100644 --- a/Examples/UIExplorer/UIExplorerUnitTests/RCTBridgeTests.m +++ b/Examples/UIExplorer/UIExplorerUnitTests/RCTBridgeTests.m @@ -241,10 +241,10 @@ RCT_EXPORT_MODULE(TestModule) NSString *injectedStuff; RUN_RUNLOOP_WHILE(!(injectedStuff = _jsExecutor.injectedStuff[@"__fbBatchedBridgeConfig"])); XCTAssertNotNil(injectedStuff); - + __block NSNumber *testModuleID = nil; __block NSNumber *testMethodID = nil; - + NSArray *remoteModuleConfig = RCTJSONParse(injectedStuff, NULL)[@"remoteModuleConfig"]; [remoteModuleConfig enumerateObjectsUsingBlock:^(id moduleConfig, NSUInteger i, __unused BOOL *stop) { if ([moduleConfig isKindOfClass:[NSArray class]] && [moduleConfig[0] isEqualToString:@"UnregisteredTestModule"]) { @@ -253,15 +253,15 @@ RCT_EXPORT_MODULE(TestModule) *stop = YES; } }]; - + XCTAssertNotNil(testModuleID); XCTAssertNotNil(testMethodID); - + NSArray *args = @[]; NSArray *buffer = @[@[testModuleID], @[testMethodID], @[args]]; - + [_bridge.batchedBridge handleBuffer:buffer]; - + dispatch_sync(_unregisteredTestModule.methodQueue, ^{ XCTAssertTrue(_unregisteredTestModule.testMethodCalled); }); diff --git a/Libraries/ActionSheetIOS/ActionSheetIOS.js b/Libraries/ActionSheetIOS/ActionSheetIOS.js index c76a1beeb..2321814fa 100644 --- a/Libraries/ActionSheetIOS/ActionSheetIOS.js +++ b/Libraries/ActionSheetIOS/ActionSheetIOS.js @@ -41,11 +41,11 @@ var ActionSheetIOS = { callback ); }, - + /** * Display the iOS share sheet. The `options` object should contain * one or both of: - * + * * - `message` (string) - a message to share * - `url` (string) - a URL to share * diff --git a/Libraries/ActionSheetIOS/RCTActionSheetManager.m b/Libraries/ActionSheetIOS/RCTActionSheetManager.m index f1eba5c0e..1be710f23 100644 --- a/Libraries/ActionSheetIOS/RCTActionSheetManager.m +++ b/Libraries/ActionSheetIOS/RCTActionSheetManager.m @@ -70,7 +70,7 @@ RCT_EXPORT_METHOD(showActionSheetWithOptions:(NSDictionary *)options while (controller.presentedViewController) { controller = controller.presentedViewController; } - + if (controller == nil) { RCTLogError(@"Tried to display action sheet but there is no application window. options: %@", options); return; diff --git a/Libraries/Components/SegmentedControlIOS/SegmentedControlIOS.ios.js b/Libraries/Components/SegmentedControlIOS/SegmentedControlIOS.ios.js index 34ef75a8a..755e32f30 100644 --- a/Libraries/Components/SegmentedControlIOS/SegmentedControlIOS.ios.js +++ b/Libraries/Components/SegmentedControlIOS/SegmentedControlIOS.ios.js @@ -60,7 +60,7 @@ var SegmentedControlIOS = React.createClass({ values: PropTypes.arrayOf(PropTypes.string), /** - * The index in `props.values` of the segment to be (pre)selected. + * The index in `props.values` of the segment to be (pre)selected. */ selectedIndex: PropTypes.number, diff --git a/Libraries/Image/ImageEditor.js b/Libraries/Image/ImageEditor.js index 4ea15349d..55746e415 100644 --- a/Libraries/Image/ImageEditor.js +++ b/Libraries/Image/ImageEditor.js @@ -28,7 +28,7 @@ type ImageCropData = { */ size: { width: number; - height: number; + height: number; }; /** * (Optional) size to scale the cropped image to. @@ -52,7 +52,7 @@ class ImageEditor { /** * Crop the image specified by the URI param. If URI points to a remote * image, it will be downloaded automatically. If the image cannot be - * loaded/downloaded, the failure callback will be called. + * loaded/downloaded, the failure callback will be called. * * If the cropping process is successful, the resultant cropped image * will be stored in the ImageStore, and the URI returned in the success diff --git a/Libraries/Image/ImageStylePropTypes.js b/Libraries/Image/ImageStylePropTypes.js index b1c5e12b1..159079e0d 100644 --- a/Libraries/Image/ImageStylePropTypes.js +++ b/Libraries/Image/ImageStylePropTypes.js @@ -30,7 +30,7 @@ var ImageStylePropTypes = { borderRadius: ReactPropTypes.number, overflow: ReactPropTypes.oneOf(['visible', 'hidden']), - /** + /** * iOS-Specific style to "tint" an image. * Changes the color of all the non-transparent pixels to the tintColor. * @platform ios @@ -44,7 +44,7 @@ var ImageStylePropTypes = { * implementation of rounded corners: * - Certain resize modes, such as 'contain' * - Animated GIFs - * + * * A typical way to use this prop is with images displayed on a solid * background and setting the `overlayColor` to the same color * as the background. diff --git a/Libraries/RCTTest/FBSnapshotTestCase/FBSnapshotTestController.m b/Libraries/RCTTest/FBSnapshotTestCase/FBSnapshotTestController.m index 7d7d32947..088569319 100644 --- a/Libraries/RCTTest/FBSnapshotTestCase/FBSnapshotTestController.m +++ b/Libraries/RCTTest/FBSnapshotTestCase/FBSnapshotTestController.m @@ -347,7 +347,7 @@ typedef NS_ENUM(NSUInteger, FBTestSnapshotFileNameType) { - (UIImage *)_snapshotViewOrLayer:(id)viewOrLayer { CALayer *layer = nil; - + if ([viewOrLayer isKindOfClass:[UIView class]]) { return [self _renderView:viewOrLayer]; } else if ([viewOrLayer isKindOfClass:[CALayer class]]) { @@ -370,19 +370,19 @@ typedef NS_ENUM(NSUInteger, FBTestSnapshotFileNameType) { UIGraphicsBeginImageContextWithOptions(bounds.size, NO, 0); CGContextRef context = UIGraphicsGetCurrentContext(); NSAssert1(context, @"Could not generate context for layer %@", layer); - + CGContextSaveGState(context); { [layer renderInContext:context]; } CGContextRestoreGState(context); - + UIImage *snapshot = UIGraphicsGetImageFromCurrentImageContext(); UIGraphicsEndImageContext(); - + return snapshot; } - + - (UIImage *)_renderView:(UIView *)view { [view layoutIfNeeded]; diff --git a/Libraries/RCTTest/FBSnapshotTestCase/UIImage+Compare.m b/Libraries/RCTTest/FBSnapshotTestCase/UIImage+Compare.m index 0bd83ba3b..24635509d 100644 --- a/Libraries/RCTTest/FBSnapshotTestCase/UIImage+Compare.m +++ b/Libraries/RCTTest/FBSnapshotTestCase/UIImage+Compare.m @@ -35,7 +35,7 @@ - (BOOL)compareWithImage:(UIImage *)image { NSAssert(CGSizeEqualToSize(self.size, image.size), @"Images must be same size."); - + // The images have the equal size, so we could use the smallest amount of bytes because of byte padding size_t minBytesPerRow = MIN(CGImageGetBytesPerRow(self.CGImage), CGImageGetBytesPerRow(image.CGImage)); size_t referenceImageSizeBytes = CGImageGetHeight(self.CGImage) * minBytesPerRow; @@ -47,7 +47,7 @@ free(imagePixels); return NO; } - + CGContextRef referenceImageContext = CGBitmapContextCreate(referenceImagePixels, CGImageGetWidth(self.CGImage), CGImageGetHeight(self.CGImage), @@ -64,11 +64,11 @@ CGImageGetColorSpace(image.CGImage), (CGBitmapInfo)kCGImageAlphaPremultipliedLast ); - + CGFloat scaleFactor = [UIScreen mainScreen].scale; CGContextScaleCTM(referenceImageContext, scaleFactor, scaleFactor); CGContextScaleCTM(imageContext, scaleFactor, scaleFactor); - + if (!referenceImageContext || !imageContext) { CGContextRelease(referenceImageContext); CGContextRelease(imageContext); @@ -76,12 +76,12 @@ free(imagePixels); return NO; } - + CGContextDrawImage(referenceImageContext, CGRectMake(0.0f, 0.0f, self.size.width, self.size.height), self.CGImage); CGContextDrawImage(imageContext, CGRectMake(0.0f, 0.0f, image.size.width, image.size.height), image.CGImage); CGContextRelease(referenceImageContext); CGContextRelease(imageContext); - + BOOL imageEqual = (memcmp(referenceImagePixels, imagePixels, referenceImageSizeBytes) == 0); free(referenceImagePixels); free(imagePixels); diff --git a/Libraries/Text/RCTTextView.m b/Libraries/Text/RCTTextView.m index b9c6a74b2..07eb7a37a 100644 --- a/Libraries/Text/RCTTextView.m +++ b/Libraries/Text/RCTTextView.m @@ -291,7 +291,7 @@ RCT_NOT_IMPLEMENTED(- (instancetype)initWithCoder:(NSCoder *)aDecoder) if (textView.textWasPasted) { textView.textWasPasted = NO; } else { - + [_eventDispatcher sendTextEventWithType:RCTTextEventTypeKeyPress reactTag:self.reactTag text:nil diff --git a/Libraries/Utilities/UIManager.js b/Libraries/Utilities/UIManager.js index efd8ce5b0..d5450d158 100644 --- a/Libraries/Utilities/UIManager.js +++ b/Libraries/Utilities/UIManager.js @@ -49,7 +49,7 @@ const _takeSnapshot = UIManager.takeSnapshot; * Capture an image of the screen, window or an individual view. The image * will be stored in a temporary file that will only exist for as long as the * app is running. - * + * * The `view` argument can be the literal string `window` if you want to * capture the entire window, or it can be a reference to a specific * React Native component. diff --git a/React/Layout/Layout.c b/React/Layout/Layout.c index 33afdb0d8..b998c9efa 100644 --- a/React/Layout/Layout.c +++ b/React/Layout/Layout.c @@ -601,7 +601,7 @@ static void layoutNodeImpl(css_node_t *node, float parentMaxWidth, float parentM if (isRowUndefined || isColumnUndefined) { css_dim_t measureDim = node->measure( node->context, - + width, height ); diff --git a/ReactAndroid/DevExperience.md b/ReactAndroid/DevExperience.md index ccb1f629f..996152908 100644 --- a/ReactAndroid/DevExperience.md +++ b/ReactAndroid/DevExperience.md @@ -9,7 +9,7 @@ Make sure you have the following installed: - Android Support Repository 17 (for Android Support Library) Follow steps on https://github.com/facebook/react-native/blob/master/react-native-cli/CONTRIBUTING.md, but be sure to bump the version of react-native in package.json to some version > 0.9 (latest published npm version) or set up proxying properly for react-native - + - From the react-native-android repo: - `./gradlew :ReactAndroid:installArchives` - *Assuming you already have android-jsc installed to local maven repo, no steps included here* diff --git a/ReactAndroid/release.gradle b/ReactAndroid/release.gradle index e5b34d73c..32e8df591 100644 --- a/ReactAndroid/release.gradle +++ b/ReactAndroid/release.gradle @@ -119,7 +119,7 @@ afterEvaluate { project -> repositories.mavenDeployer { // Deploy to react-native/android, ready to publish to npm repository url: "file://${projectDir}/../android" - + configureReactNativePom pom } } diff --git a/ReactAndroid/src/main/java/com/facebook/csslayout/LayoutEngine.java b/ReactAndroid/src/main/java/com/facebook/csslayout/LayoutEngine.java index ebef5c3a4..0e6f3f675 100644 --- a/ReactAndroid/src/main/java/com/facebook/csslayout/LayoutEngine.java +++ b/ReactAndroid/src/main/java/com/facebook/csslayout/LayoutEngine.java @@ -227,19 +227,19 @@ public class LayoutEngine { float parentMaxHeight, CSSDirection parentDirection) { /** START_GENERATED **/ - + CSSDirection direction = resolveDirection(node, parentDirection); int mainAxis = resolveAxis(getFlexDirection(node), direction); int crossAxis = getCrossFlexDirection(mainAxis, direction); int resolvedRowAxis = resolveAxis(CSS_FLEX_DIRECTION_ROW, direction); - + // Handle width and height style attributes setDimensionFromStyle(node, mainAxis); setDimensionFromStyle(node, crossAxis); - + // Set the resolved resolution in the node's layout node.layout.direction = direction; - + // The position is set by the parent, but we need to complete it with a // delta composed of the margin and left/top/right/bottom node.layout.position[leading[mainAxis]] += node.style.margin.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + @@ -250,16 +250,16 @@ public class LayoutEngine { getRelativePosition(node, crossAxis); node.layout.position[trailing[crossAxis]] += node.style.margin.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis]) + getRelativePosition(node, crossAxis); - + // Inline immutable values from the target node to avoid excessive method // invocations during the layout calculation. int childCount = node.getChildCount(); float paddingAndBorderAxisResolvedRow = ((node.style.padding.getWithFallback(leadingSpacing[resolvedRowAxis], leading[resolvedRowAxis]) + node.style.border.getWithFallback(leadingSpacing[resolvedRowAxis], leading[resolvedRowAxis])) + (node.style.padding.getWithFallback(trailingSpacing[resolvedRowAxis], trailing[resolvedRowAxis]) + node.style.border.getWithFallback(trailingSpacing[resolvedRowAxis], trailing[resolvedRowAxis]))); float paddingAndBorderAxisColumn = ((node.style.padding.getWithFallback(leadingSpacing[CSS_FLEX_DIRECTION_COLUMN], leading[CSS_FLEX_DIRECTION_COLUMN]) + node.style.border.getWithFallback(leadingSpacing[CSS_FLEX_DIRECTION_COLUMN], leading[CSS_FLEX_DIRECTION_COLUMN])) + (node.style.padding.getWithFallback(trailingSpacing[CSS_FLEX_DIRECTION_COLUMN], trailing[CSS_FLEX_DIRECTION_COLUMN]) + node.style.border.getWithFallback(trailingSpacing[CSS_FLEX_DIRECTION_COLUMN], trailing[CSS_FLEX_DIRECTION_COLUMN]))); - + if (isMeasureDefined(node)) { boolean isResolvedRowDimDefined = (!Float.isNaN(node.layout.dimensions[dim[resolvedRowAxis]]) && node.layout.dimensions[dim[resolvedRowAxis]] >= 0.0); - + float width = CSSConstants.UNDEFINED; if ((!Float.isNaN(node.style.dimensions[dim[resolvedRowAxis]]) && node.style.dimensions[dim[resolvedRowAxis]] >= 0.0)) { width = node.style.dimensions[DIMENSION_WIDTH]; @@ -270,7 +270,7 @@ public class LayoutEngine { (node.style.margin.getWithFallback(leadingSpacing[resolvedRowAxis], leading[resolvedRowAxis]) + node.style.margin.getWithFallback(trailingSpacing[resolvedRowAxis], trailing[resolvedRowAxis])); } width -= paddingAndBorderAxisResolvedRow; - + float height = CSSConstants.UNDEFINED; if ((!Float.isNaN(node.style.dimensions[dim[CSS_FLEX_DIRECTION_COLUMN]]) && node.style.dimensions[dim[CSS_FLEX_DIRECTION_COLUMN]] >= 0.0)) { height = node.style.dimensions[DIMENSION_HEIGHT]; @@ -281,18 +281,18 @@ public class LayoutEngine { (node.style.margin.getWithFallback(leadingSpacing[resolvedRowAxis], leading[resolvedRowAxis]) + node.style.margin.getWithFallback(trailingSpacing[resolvedRowAxis], trailing[resolvedRowAxis])); } height -= ((node.style.padding.getWithFallback(leadingSpacing[CSS_FLEX_DIRECTION_COLUMN], leading[CSS_FLEX_DIRECTION_COLUMN]) + node.style.border.getWithFallback(leadingSpacing[CSS_FLEX_DIRECTION_COLUMN], leading[CSS_FLEX_DIRECTION_COLUMN])) + (node.style.padding.getWithFallback(trailingSpacing[CSS_FLEX_DIRECTION_COLUMN], trailing[CSS_FLEX_DIRECTION_COLUMN]) + node.style.border.getWithFallback(trailingSpacing[CSS_FLEX_DIRECTION_COLUMN], trailing[CSS_FLEX_DIRECTION_COLUMN]))); - + // We only need to give a dimension for the text if we haven't got any // for it computed yet. It can either be from the style attribute or because // the element is flexible. boolean isRowUndefined = !(!Float.isNaN(node.style.dimensions[dim[resolvedRowAxis]]) && node.style.dimensions[dim[resolvedRowAxis]] >= 0.0) && !isResolvedRowDimDefined; boolean isColumnUndefined = !(!Float.isNaN(node.style.dimensions[dim[CSS_FLEX_DIRECTION_COLUMN]]) && node.style.dimensions[dim[CSS_FLEX_DIRECTION_COLUMN]] >= 0.0) && Float.isNaN(node.layout.dimensions[dim[CSS_FLEX_DIRECTION_COLUMN]]); - + // Let's not measure the text if we already know both dimensions if (isRowUndefined || isColumnUndefined) { MeasureOutput measureDim = node.measure( - + layoutContext.measureOutput, width, height @@ -310,33 +310,33 @@ public class LayoutEngine { return; } } - + boolean isNodeFlexWrap = (node.style.flexWrap == CSSWrap.WRAP); - + CSSJustify justifyContent = node.style.justifyContent; - + float leadingPaddingAndBorderMain = (node.style.padding.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + node.style.border.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis])); float leadingPaddingAndBorderCross = (node.style.padding.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]) + node.style.border.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis])); float paddingAndBorderAxisMain = ((node.style.padding.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + node.style.border.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis])) + (node.style.padding.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]) + node.style.border.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]))); float paddingAndBorderAxisCross = ((node.style.padding.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]) + node.style.border.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis])) + (node.style.padding.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis]) + node.style.border.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis]))); - + boolean isMainDimDefined = (!Float.isNaN(node.layout.dimensions[dim[mainAxis]]) && node.layout.dimensions[dim[mainAxis]] >= 0.0); boolean isCrossDimDefined = (!Float.isNaN(node.layout.dimensions[dim[crossAxis]]) && node.layout.dimensions[dim[crossAxis]] >= 0.0); boolean isMainRowDirection = (mainAxis == CSS_FLEX_DIRECTION_ROW || mainAxis == CSS_FLEX_DIRECTION_ROW_REVERSE); - + int i; int ii; CSSNode child; int axis; - + CSSNode firstAbsoluteChild = null; CSSNode currentAbsoluteChild = null; - + float definedMainDim = CSSConstants.UNDEFINED; if (isMainDimDefined) { definedMainDim = node.layout.dimensions[dim[mainAxis]] - paddingAndBorderAxisMain; } - + // We want to execute the next two loops one per line with flex-wrap int startLine = 0; int endLine = 0; @@ -348,19 +348,19 @@ public class LayoutEngine { int linesCount = 0; while (endLine < childCount) { // Layout non flexible children and count children by type - + // mainContentDim is accumulation of the dimensions and margin of all the // non flexible children. This will be used in order to either set the // dimensions of the node if none already exist, or to compute the // remaining space left for the flexible children. float mainContentDim = 0; - + // There are three kind of children, non flexible, flexible and absolute. // We need to know how many there are in order to distribute the space. int flexibleChildrenCount = 0; float totalFlexible = 0; int nonFlexibleChildrenCount = 0; - + // Use the line loop to position children in the main axis for as long // as they are using a simple stacking behaviour. Children that are // immediately stacked in the initial loop will not be touched again @@ -369,31 +369,31 @@ public class LayoutEngine { (isMainDimDefined && justifyContent == CSSJustify.FLEX_START) || (!isMainDimDefined && justifyContent != CSSJustify.CENTER); int firstComplexMain = (isSimpleStackMain ? childCount : startLine); - + // Use the initial line loop to position children in the cross axis for // as long as they are relatively positioned with alignment STRETCH or // FLEX_START. Children that are immediately stacked in the initial loop // will not be touched again in . boolean isSimpleStackCross = true; int firstComplexCross = childCount; - + CSSNode firstFlexChild = null; CSSNode currentFlexChild = null; - + float mainDim = leadingPaddingAndBorderMain; float crossDim = 0; - + float maxWidth = CSSConstants.UNDEFINED; float maxHeight = CSSConstants.UNDEFINED; for (i = startLine; i < childCount; ++i) { child = node.getChildAt(i); child.lineIndex = linesCount; - + child.nextAbsoluteChild = null; child.nextFlexChild = null; - + CSSAlign alignItem = getAlignItem(node, child); - + // Pre-fill cross axis dimensions when the child is using stretch before // we call the recursive layout pass if (alignItem == CSSAlign.STRETCH && @@ -416,7 +416,7 @@ public class LayoutEngine { currentAbsoluteChild.nextAbsoluteChild = child; } currentAbsoluteChild = child; - + // Pre-fill dimensions when using absolute position and both offsets for the axis are defined (either both // left and right or top and bottom). for (ii = 0; ii < 2; ii++) { @@ -437,15 +437,15 @@ public class LayoutEngine { } } } - + float nextContentDim = 0; - + // It only makes sense to consider a child flexible if we have a computed // dimension for the node. if (isMainDimDefined && (child.style.positionType == CSSPositionType.RELATIVE && child.style.flex > 0)) { flexibleChildrenCount++; totalFlexible += child.style.flex; - + // Store a private linked list of flexible children so that we can // efficiently traverse them later. if (firstFlexChild == null) { @@ -455,18 +455,18 @@ public class LayoutEngine { currentFlexChild.nextFlexChild = child; } currentFlexChild = child; - + // Even if we don't know its exact size yet, we already know the padding, // border and margin. We'll use this partial information, which represents // the smallest possible size for the child, to compute the remaining // available space. nextContentDim = ((child.style.padding.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + child.style.border.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis])) + (child.style.padding.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]) + child.style.border.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]))) + (child.style.margin.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + child.style.margin.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis])); - + } else { maxWidth = CSSConstants.UNDEFINED; maxHeight = CSSConstants.UNDEFINED; - + if (!isMainRowDirection) { if ((!Float.isNaN(node.layout.dimensions[dim[resolvedRowAxis]]) && node.layout.dimensions[dim[resolvedRowAxis]] >= 0.0)) { maxWidth = node.layout.dimensions[dim[resolvedRowAxis]] - @@ -486,12 +486,12 @@ public class LayoutEngine { paddingAndBorderAxisColumn; } } - + // This is the main recursive call. We layout non flexible children. if (alreadyComputedNextLayout == 0) { layoutNode(layoutContext, child, maxWidth, maxHeight, direction); } - + // Absolute positioned elements do not take part of the layout, so we // don't use them to compute mainContentDim if (child.style.positionType == CSSPositionType.RELATIVE) { @@ -500,7 +500,7 @@ public class LayoutEngine { nextContentDim = (child.layout.dimensions[dim[mainAxis]] + child.style.margin.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + child.style.margin.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis])); } } - + // The element we are about to add would make us go to the next line if (isNodeFlexWrap && isMainDimDefined && @@ -512,7 +512,7 @@ public class LayoutEngine { alreadyComputedNextLayout = 1; break; } - + // Disable simple stacking in the main axis for the current line as // we found a non-trivial child. The remaining children will be laid out // in . @@ -521,7 +521,7 @@ public class LayoutEngine { isSimpleStackMain = false; firstComplexMain = i; } - + // Disable simple stacking in the cross axis for the current line as // we found a non-trivial child. The remaining children will be laid out // in . @@ -532,37 +532,37 @@ public class LayoutEngine { isSimpleStackCross = false; firstComplexCross = i; } - + if (isSimpleStackMain) { child.layout.position[pos[mainAxis]] += mainDim; if (isMainDimDefined) { child.layout.position[trailing[mainAxis]] = node.layout.dimensions[dim[mainAxis]] - child.layout.dimensions[dim[mainAxis]] - child.layout.position[pos[mainAxis]]; } - + mainDim += (child.layout.dimensions[dim[mainAxis]] + child.style.margin.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + child.style.margin.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis])); crossDim = Math.max(crossDim, boundAxis(child, crossAxis, (child.layout.dimensions[dim[crossAxis]] + child.style.margin.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]) + child.style.margin.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis])))); } - + if (isSimpleStackCross) { child.layout.position[pos[crossAxis]] += linesCrossDim + leadingPaddingAndBorderCross; if (isCrossDimDefined) { child.layout.position[trailing[crossAxis]] = node.layout.dimensions[dim[crossAxis]] - child.layout.dimensions[dim[crossAxis]] - child.layout.position[pos[crossAxis]]; } } - + alreadyComputedNextLayout = 0; mainContentDim += nextContentDim; endLine = i + 1; } - + // Layout flexible children and allocate empty space - + // In order to position the elements in the main axis, we have two // controls. The space between the beginning and the first element // and the space between each two elements. float leadingMainDim = 0; float betweenMainDim = 0; - + // The remaining available space that needs to be allocated float remainingMainDim = 0; if (isMainDimDefined) { @@ -570,14 +570,14 @@ public class LayoutEngine { } else { remainingMainDim = Math.max(mainContentDim, 0) - mainContentDim; } - + // If there are flexible children in the mix, they are going to fill the // remaining space if (flexibleChildrenCount != 0) { float flexibleMainDim = remainingMainDim / totalFlexible; float baseMainDim; float boundMainDim; - + // If the flex share of remaining space doesn't meet min/max bounds, // remove this child from flex calculations. currentFlexChild = firstFlexChild; @@ -585,22 +585,22 @@ public class LayoutEngine { baseMainDim = flexibleMainDim * currentFlexChild.style.flex + ((currentFlexChild.style.padding.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + currentFlexChild.style.border.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis])) + (currentFlexChild.style.padding.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]) + currentFlexChild.style.border.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]))); boundMainDim = boundAxis(currentFlexChild, mainAxis, baseMainDim); - + if (baseMainDim != boundMainDim) { remainingMainDim -= boundMainDim; totalFlexible -= currentFlexChild.style.flex; } - + currentFlexChild = currentFlexChild.nextFlexChild; } flexibleMainDim = remainingMainDim / totalFlexible; - + // The non flexible children can overflow the container, in this case // we should just assume that there is no space available. if (flexibleMainDim < 0) { flexibleMainDim = 0; } - + currentFlexChild = firstFlexChild; while (currentFlexChild != null) { // At this point we know the final size of the element in the main @@ -609,7 +609,7 @@ public class LayoutEngine { flexibleMainDim * currentFlexChild.style.flex + ((currentFlexChild.style.padding.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis]) + currentFlexChild.style.border.getWithFallback(leadingSpacing[mainAxis], leading[mainAxis])) + (currentFlexChild.style.padding.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]) + currentFlexChild.style.border.getWithFallback(trailingSpacing[mainAxis], trailing[mainAxis]))) ); - + maxWidth = CSSConstants.UNDEFINED; if ((!Float.isNaN(node.layout.dimensions[dim[resolvedRowAxis]]) && node.layout.dimensions[dim[resolvedRowAxis]] >= 0.0)) { maxWidth = node.layout.dimensions[dim[resolvedRowAxis]] - @@ -628,15 +628,15 @@ public class LayoutEngine { (node.style.margin.getWithFallback(leadingSpacing[CSS_FLEX_DIRECTION_COLUMN], leading[CSS_FLEX_DIRECTION_COLUMN]) + node.style.margin.getWithFallback(trailingSpacing[CSS_FLEX_DIRECTION_COLUMN], trailing[CSS_FLEX_DIRECTION_COLUMN])) - paddingAndBorderAxisColumn; } - + // And we recursively call the layout algorithm for this child layoutNode(layoutContext, currentFlexChild, maxWidth, maxHeight, direction); - + child = currentFlexChild; currentFlexChild = currentFlexChild.nextFlexChild; child.nextFlexChild = null; } - + // We use justifyContent to figure out how to allocate the remaining // space available } else if (justifyContent != CSSJustify.FLEX_START) { @@ -659,18 +659,18 @@ public class LayoutEngine { leadingMainDim = betweenMainDim / 2; } } - + // Position elements in the main axis and compute dimensions - + // At this point, all the children have their dimensions set. We need to // find their position. In order to do that, we accumulate data in // variables that are also useful to compute the total dimensions of the // container! mainDim += leadingMainDim; - + for (i = firstComplexMain; i < endLine; ++i) { child = node.getChildAt(i); - + if (child.style.positionType == CSSPositionType.ABSOLUTE && !Float.isNaN(child.style.position[leading[mainAxis]])) { // In case the child is position absolute and has left/top being @@ -683,12 +683,12 @@ public class LayoutEngine { // If the child is position absolute (without top/left) or relative, // we put it at the current accumulated offset. child.layout.position[pos[mainAxis]] += mainDim; - + // Define the trailing position accordingly. if (isMainDimDefined) { child.layout.position[trailing[mainAxis]] = node.layout.dimensions[dim[mainAxis]] - child.layout.dimensions[dim[mainAxis]] - child.layout.position[pos[mainAxis]]; } - + // Now that we placed the element, we need to update the variables // We only need to do that for relative elements. Absolute elements // do not take part in that phase. @@ -702,7 +702,7 @@ public class LayoutEngine { } } } - + float containerCrossAxis = node.layout.dimensions[dim[crossAxis]]; if (!isCrossDimDefined) { containerCrossAxis = Math.max( @@ -713,11 +713,11 @@ public class LayoutEngine { paddingAndBorderAxisCross ); } - + // Position elements in the cross axis for (i = firstComplexCross; i < endLine; ++i) { child = node.getChildAt(i); - + if (child.style.positionType == CSSPositionType.ABSOLUTE && !Float.isNaN(child.style.position[leading[crossAxis]])) { // In case the child is absolutely positionned and has a @@ -726,10 +726,10 @@ public class LayoutEngine { child.layout.position[pos[crossAxis]] = (Float.isNaN(child.style.position[leading[crossAxis]]) ? 0 : child.style.position[leading[crossAxis]]) + node.style.border.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]) + child.style.margin.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]); - + } else { float leadingCrossDim = leadingPaddingAndBorderCross; - + // For a relative children, we're either using alignItems (parent) or // alignSelf (child) in order to determine the position in the cross axis if (child.style.positionType == CSSPositionType.RELATIVE) { @@ -748,7 +748,7 @@ public class LayoutEngine { // You never want to go smaller than padding ((child.style.padding.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]) + child.style.border.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis])) + (child.style.padding.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis]) + child.style.border.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis]))) ); - + // If the size has changed, and this child has children we need to re-layout this child if (dimCrossAxis != child.layout.dimensions[dim[crossAxis]] && child.getChildCount() > 0) { // Reset child margins before re-layout as they are added back in layoutNode and would be doubled @@ -760,7 +760,7 @@ public class LayoutEngine { getRelativePosition(child, crossAxis); child.layout.position[trailing[crossAxis]] -= child.style.margin.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis]) + getRelativePosition(child, crossAxis); - + layoutNode(layoutContext, child, maxWidth, maxHeight, direction); } } @@ -769,7 +769,7 @@ public class LayoutEngine { // dimensions+margin. float remainingCrossDim = containerCrossAxis - paddingAndBorderAxisCross - (child.layout.dimensions[dim[crossAxis]] + child.style.margin.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]) + child.style.margin.getWithFallback(trailingSpacing[crossAxis], trailing[crossAxis])); - + if (alignItem == CSSAlign.CENTER) { leadingCrossDim += remainingCrossDim / 2; } else { // CSSAlign.FLEX_END @@ -777,23 +777,23 @@ public class LayoutEngine { } } } - + // And we apply the position child.layout.position[pos[crossAxis]] += linesCrossDim + leadingCrossDim; - + // Define the trailing position accordingly. if (isCrossDimDefined) { child.layout.position[trailing[crossAxis]] = node.layout.dimensions[dim[crossAxis]] - child.layout.dimensions[dim[crossAxis]] - child.layout.position[pos[crossAxis]]; } } } - + linesCrossDim += crossDim; linesMainDim = Math.max(linesMainDim, mainDim); linesCount += 1; startLine = endLine; } - + // // // Note(prenaux): More than one line, we need to layout the crossAxis @@ -811,10 +811,10 @@ public class LayoutEngine { float nodeCrossAxisInnerSize = node.layout.dimensions[dim[crossAxis]] - paddingAndBorderAxisCross; float remainingAlignContentDim = nodeCrossAxisInnerSize - linesCrossDim; - + float crossDimLead = 0; float currentLead = leadingPaddingAndBorderCross; - + CSSAlign alignContent = node.style.alignContent; if (alignContent == CSSAlign.FLEX_END) { currentLead += remainingAlignContentDim; @@ -825,11 +825,11 @@ public class LayoutEngine { crossDimLead = (remainingAlignContentDim / linesCount); } } - + int endIndex = 0; for (i = 0; i < linesCount; ++i) { int startIndex = endIndex; - + // compute the line's height and find the endIndex float lineHeight = 0; for (ii = startIndex; ii < childCount; ++ii) { @@ -849,13 +849,13 @@ public class LayoutEngine { } endIndex = ii; lineHeight += crossDimLead; - + for (ii = startIndex; ii < endIndex; ++ii) { child = node.getChildAt(ii); if (child.style.positionType != CSSPositionType.RELATIVE) { continue; } - + CSSAlign alignContentAlignItem = getAlignItem(node, child); if (alignContentAlignItem == CSSAlign.FLEX_START) { child.layout.position[pos[crossAxis]] = currentLead + child.style.margin.getWithFallback(leadingSpacing[crossAxis], leading[crossAxis]); @@ -870,14 +870,14 @@ public class LayoutEngine { // (auto) crossAxis dimension. } } - + currentLead += lineHeight; } } - + boolean needsMainTrailingPos = false; boolean needsCrossTrailingPos = false; - + // If the user didn't specify a width or height, and it has not been set // by the container, then we set it via the children. if (!isMainDimDefined) { @@ -888,13 +888,13 @@ public class LayoutEngine { // We can never assign a width smaller than the padding and borders paddingAndBorderAxisMain ); - + if (mainAxis == CSS_FLEX_DIRECTION_ROW_REVERSE || mainAxis == CSS_FLEX_DIRECTION_COLUMN_REVERSE) { needsMainTrailingPos = true; } } - + if (!isCrossDimDefined) { node.layout.dimensions[dim[crossAxis]] = Math.max( // For the cross dim, we add both sides at the end because the value @@ -903,28 +903,28 @@ public class LayoutEngine { boundAxis(node, crossAxis, linesCrossDim + paddingAndBorderAxisCross), paddingAndBorderAxisCross ); - + if (crossAxis == CSS_FLEX_DIRECTION_ROW_REVERSE || crossAxis == CSS_FLEX_DIRECTION_COLUMN_REVERSE) { needsCrossTrailingPos = true; } } - + // Set trailing position if necessary if (needsMainTrailingPos || needsCrossTrailingPos) { for (i = 0; i < childCount; ++i) { child = node.getChildAt(i); - + if (needsMainTrailingPos) { child.layout.position[trailing[mainAxis]] = node.layout.dimensions[dim[mainAxis]] - child.layout.dimensions[dim[mainAxis]] - child.layout.position[pos[mainAxis]]; } - + if (needsCrossTrailingPos) { child.layout.position[trailing[crossAxis]] = node.layout.dimensions[dim[crossAxis]] - child.layout.dimensions[dim[crossAxis]] - child.layout.position[pos[crossAxis]]; } } } - + // Calculate dimensions for absolutely positioned elements currentAbsoluteChild = firstAbsoluteChild; while (currentAbsoluteChild != null) { @@ -932,7 +932,7 @@ public class LayoutEngine { // the axis are defined (either both left and right or top and bottom). for (ii = 0; ii < 2; ii++) { axis = (ii != 0) ? CSS_FLEX_DIRECTION_ROW : CSS_FLEX_DIRECTION_COLUMN; - + if ((!Float.isNaN(node.layout.dimensions[dim[axis]]) && node.layout.dimensions[dim[axis]] >= 0.0) && !(!Float.isNaN(currentAbsoluteChild.style.dimensions[dim[axis]]) && currentAbsoluteChild.style.dimensions[dim[axis]] >= 0.0) && !Float.isNaN(currentAbsoluteChild.style.position[leading[axis]]) && @@ -948,7 +948,7 @@ public class LayoutEngine { ((currentAbsoluteChild.style.padding.getWithFallback(leadingSpacing[axis], leading[axis]) + currentAbsoluteChild.style.border.getWithFallback(leadingSpacing[axis], leading[axis])) + (currentAbsoluteChild.style.padding.getWithFallback(trailingSpacing[axis], trailing[axis]) + currentAbsoluteChild.style.border.getWithFallback(trailingSpacing[axis], trailing[axis]))) ); } - + if (!Float.isNaN(currentAbsoluteChild.style.position[trailing[axis]]) && !!Float.isNaN(currentAbsoluteChild.style.position[leading[axis]])) { currentAbsoluteChild.layout.position[leading[axis]] = @@ -957,7 +957,7 @@ public class LayoutEngine { (Float.isNaN(currentAbsoluteChild.style.position[trailing[axis]]) ? 0 : currentAbsoluteChild.style.position[trailing[axis]]); } } - + child = currentAbsoluteChild; currentAbsoluteChild = currentAbsoluteChild.nextAbsoluteChild; child.nextAbsoluteChild = null; diff --git a/ReactAndroid/src/main/java/com/facebook/jni/Prerequisites.java b/ReactAndroid/src/main/java/com/facebook/jni/Prerequisites.java index b669e546b..74d137aa5 100644 --- a/ReactAndroid/src/main/java/com/facebook/jni/Prerequisites.java +++ b/ReactAndroid/src/main/java/com/facebook/jni/Prerequisites.java @@ -33,7 +33,7 @@ public class Prerequisites { } // Code is simplified version of getDetectedVersion() - // from cts/tests/tests/graphics/src/android/opengl/cts/OpenGlEsVersionTest.java + // from cts/tests/tests/graphics/src/android/opengl/cts/OpenGlEsVersionTest.java static public boolean supportsOpenGL20() { EGL10 egl = (EGL10) EGLContext.getEGL(); EGLDisplay display = egl.eglGetDisplay(EGL10.EGL_DEFAULT_DISPLAY); diff --git a/ReactAndroid/src/main/java/com/facebook/react/modules/toast/ToastModule.java b/ReactAndroid/src/main/java/com/facebook/react/modules/toast/ToastModule.java index ea6cb471f..af128a961 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/modules/toast/ToastModule.java +++ b/ReactAndroid/src/main/java/com/facebook/react/modules/toast/ToastModule.java @@ -51,7 +51,7 @@ public class ToastModule extends ReactContextBaseJavaModule { UiThreadUtil.runOnUiThread(new Runnable() { @Override public void run(){ - Toast.makeText(getReactApplicationContext(), message, duration).show(); + Toast.makeText(getReactApplicationContext(), message, duration).show(); } }); } diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java index 9a697ce3a..694250539 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java @@ -64,7 +64,7 @@ public abstract class BaseViewManager= Build.VERSION_CODES.LOLLIPOP) { diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIImplementation.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIImplementation.java index 443c8a2a5..b4b724240 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIImplementation.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIImplementation.java @@ -731,7 +731,7 @@ public class UIImplementation { absoluteY, mOperationsQueue, mNativeViewHierarchyOptimizer); - + // notify JS about layout event if requested if (cssNode.shouldNotifyOnLayout()) { eventDispatcher.dispatchEvent( diff --git a/ReactAndroid/src/main/java/com/facebook/react/views/webview/ReactWebViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/views/webview/ReactWebViewManager.java index b36860104..08b8a7287 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/views/webview/ReactWebViewManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/views/webview/ReactWebViewManager.java @@ -267,7 +267,7 @@ public class ReactWebViewManager extends SimpleViewManager { public void setDomStorageEnabled(WebView view, boolean enabled) { view.getSettings().setDomStorageEnabled(enabled); } - + @ReactProp(name = "userAgent") public void setUserAgent(WebView view, @Nullable String userAgent) { diff --git a/ReactAndroid/src/main/jni/first-party/fb/include/fb/log.h b/ReactAndroid/src/main/jni/first-party/fb/include/fb/log.h index 4e08b558d..5bac939dd 100644 --- a/ReactAndroid/src/main/jni/first-party/fb/include/fb/log.h +++ b/ReactAndroid/src/main/jni/first-party/fb/include/fb/log.h @@ -44,13 +44,13 @@ #ifdef __cplusplus extern "C" { #endif - + #ifdef ANDROID #include #else // These declarations are needed for our internal use even on non-Android builds. // (they are borrowed from ) - + /* * Android log priority values, in ascending priority order. */ @@ -65,12 +65,12 @@ extern "C" { ANDROID_LOG_FATAL, ANDROID_LOG_SILENT, /* only for SetMinPriority(); must be last */ } android_LogPriority; - + /* * Send a simple string to the log. */ int __android_log_write(int prio, const char *tag, const char *text); - + /* * Send a formatted string to the log, used like printf(fmt,...) */ diff --git a/ReactAndroid/src/main/jni/first-party/fbgloginit/glog_init.cpp b/ReactAndroid/src/main/jni/first-party/fbgloginit/glog_init.cpp index 771516a2d..e9688372c 100644 --- a/ReactAndroid/src/main/jni/first-party/fbgloginit/glog_init.cpp +++ b/ReactAndroid/src/main/jni/first-party/fbgloginit/glog_init.cpp @@ -103,7 +103,7 @@ static void lastResort(const char* tag, const char* msg, const char* arg = nullp } else { __android_log_print(ANDROID_LOG_ERROR, tag, "%s: %s", msg, arg); } -#else +#else std::cerr << msg; if (arg) { std::cerr << ": " << arg; diff --git a/ReactAndroid/src/main/jni/first-party/jni/Countable.cpp b/ReactAndroid/src/main/jni/first-party/jni/Countable.cpp index 3ed52f782..6ff7efe90 100644 --- a/ReactAndroid/src/main/jni/first-party/jni/Countable.cpp +++ b/ReactAndroid/src/main/jni/first-party/jni/Countable.cpp @@ -41,7 +41,7 @@ void setCountableForJava(JNIEnv* env, jobject obj, RefPtr&& countable * * This method deletes the corresponding native object on whatever thread the method is called * on. In the common case when this is called by Countable#finalize(), this will be called on the - * system finalizer thread. If you manually call dispose on the Java object, the native object + * system finalizer thread. If you manually call dispose on the Java object, the native object * will be deleted synchronously on that thread. */ void dispose(JNIEnv* env, jobject obj) { diff --git a/ReactAndroid/src/main/jni/first-party/jni/LocalReference.h b/ReactAndroid/src/main/jni/first-party/jni/LocalReference.h index 09aa641fd..b5d9c54a1 100644 --- a/ReactAndroid/src/main/jni/first-party/jni/LocalReference.h +++ b/ReactAndroid/src/main/jni/first-party/jni/LocalReference.h @@ -27,7 +27,7 @@ struct LocalReferenceDeleter { if (localReference != nullptr) { Environment::current()->DeleteLocalRef(localReference); } - } + } }; template diff --git a/ReactAndroid/src/main/jni/react/JSCExecutor.cpp b/ReactAndroid/src/main/jni/react/JSCExecutor.cpp index a30b7107f..7b91d4026 100644 --- a/ReactAndroid/src/main/jni/react/JSCExecutor.cpp +++ b/ReactAndroid/src/main/jni/react/JSCExecutor.cpp @@ -227,7 +227,7 @@ void JSCExecutor::loadApplicationScript( #else String jsScript = String::createExpectingAscii(script); #endif - + ReactMarker::logMarker("loadApplicationScript_endStringConvert"); String jsSourceURL(sourceURL.c_str()); diff --git a/docs/EmbeddedAppIOS.md b/docs/EmbeddedAppIOS.md index f13abc851..e013ac74c 100644 --- a/docs/EmbeddedAppIOS.md +++ b/docs/EmbeddedAppIOS.md @@ -162,13 +162,13 @@ class ReactView: UIView { let rootView: RCTRootView = RCTRootView(bundleURL: NSURL(string: "http://localhost:8081/index.ios.bundle?platform=ios"), moduleName: "SimpleApp", initialProperties: nil, launchOptions: nil) - + override func layoutSubviews() { super.layoutSubviews() loadReact() } - + func loadReact () { addSubview(rootView) rootView.frame = self.bounds diff --git a/docs/GettingStartedOnLinux.md b/docs/GettingStartedOnLinux.md index ebe4d8a15..81c332d87 100644 --- a/docs/GettingStartedOnLinux.md +++ b/docs/GettingStartedOnLinux.md @@ -57,7 +57,7 @@ sudo npm install -g flow-bin ## Setting up an Android Device -Let's set up an Android device to run our starter project. +Let's set up an Android device to run our starter project. First thing is to plug in your device and check the manufacturer code by using `lsusb`, which should output something like this: @@ -65,7 +65,7 @@ First thing is to plug in your device and check the manufacturer code by using ` $ lsusb Bus 002 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 002 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub -Bus 001 Device 003: ID 22b8:2e76 Motorola PCS +Bus 001 Device 003: ID 22b8:2e76 Motorola PCS Bus 001 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub @@ -90,7 +90,7 @@ You'll see that after removing the phone, the line which has the phone model ("M From the above line, you want to grab the first four digits from the device ID: -`22b8:2e76` +`22b8:2e76` In this case, it's `22b8`. That's the identifier for Motorola. diff --git a/docs/IssueGuidelines.md b/docs/IssueGuidelines.md index bf2cc4f9a..5f95c9135 100644 --- a/docs/IssueGuidelines.md +++ b/docs/IssueGuidelines.md @@ -11,7 +11,7 @@ Examples: [#6378](https://github.com/facebook/react-native/issues/6378), [#6015] Feel free to also answer some [SO questions](stackoverflow.com/questions/tagged/react-native), you'll get rep :) ### An issue is a question that's been answered -Sometimes and issue has been resolved in the comments. +Sometimes and issue has been resolved in the comments. Comment `@facebook-github-bot answered` to close it. Example: [#6045](https://github.com/facebook/react-native/issues/6045) diff --git a/docs/JavaScriptEnvironment.md b/docs/JavaScriptEnvironment.md index 8bda77498..90605fe1a 100644 --- a/docs/JavaScriptEnvironment.md +++ b/docs/JavaScriptEnvironment.md @@ -67,7 +67,7 @@ Browser * [XMLHttpRequest, fetch](/react-native/docs/network.html#content) * [{set, clear}{Timeout, Interval, Immediate}, {request, cancel}AnimationFrame](/react-native/docs/timers.html#content) * [navigator.geolocation](/react-native/docs/geolocation.html#content) - + ES6 * [Object.assign](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign) diff --git a/docs/LinuxWindowsSupport.md b/docs/LinuxWindowsSupport.md index b1cf5f82d..720b37ac6 100644 --- a/docs/LinuxWindowsSupport.md +++ b/docs/LinuxWindowsSupport.md @@ -9,7 +9,7 @@ next: tutorial __NOTE: This guide focuses on Android development. You'll need a Mac to build iOS apps.__ -As React Native on iOS requires a Mac and most of the engineers at Facebook and contributors use Macs, support for OS X is a top priority. However, we would like to support developers using Linux and Windows too. We believe we'll get the best Linux and Windows support from people using these operating systems on a daily basis. +As React Native on iOS requires a Mac and most of the engineers at Facebook and contributors use Macs, support for OS X is a top priority. However, we would like to support developers using Linux and Windows too. We believe we'll get the best Linux and Windows support from people using these operating systems on a daily basis. Therefore, Linux and Windows support for the development environment is an ongoing community responsibility. This can mean filing issues and submitting PRs, and we'll help review and merge them. We are looking forward to your contributions and appreciate your patience. @@ -18,7 +18,7 @@ As of **version 0.14** Android development with React native is mostly possible ## What's missing on Windows On Windows the packager won't be started automatically when you run `react-native run-android`. You can start it manually using: - + cd MyAwesomeApp react-native start diff --git a/docs/Text.md b/docs/Text.md index 4c48ef1ca..d49194faa 100644 --- a/docs/Text.md +++ b/docs/Text.md @@ -23,7 +23,7 @@ Behind the scenes, this is going to be converted to a flat `NSAttributedString` ## Containers -The `` element is special relative to layout: everything inside is no longer using the flexbox layout but using text layout. This means that elements inside of a `` are no longer rectangles, but wrap when they see the end of the line. +The `` element is special relative to layout: everything inside is no longer using the flexbox layout but using text layout. This means that elements inside of a `` are no longer rectangles, but wrap when they see the end of the line. ```javascript diff --git a/docs/Tutorial.md b/docs/Tutorial.md index ec5a4d72b..1da167e2e 100644 --- a/docs/Tutorial.md +++ b/docs/Tutorial.md @@ -219,7 +219,7 @@ Add some initial state to our application so that we can check `this.state.movie ```javascript constructor(props) { - super(props); + super(props); this.state = { movies: null, }; diff --git a/local-cli/runAndroid/adb.js b/local-cli/runAndroid/adb.js index 8128c866d..46ae8b393 100644 --- a/local-cli/runAndroid/adb.js +++ b/local-cli/runAndroid/adb.js @@ -5,7 +5,7 @@ * This source code is licensed under the BSD-style license found in the * LICENSE file in the root directory of this source tree. An additional grant * of patent rights can be found in the PATENTS file in the same directory. - * + * * @flow */ @@ -18,13 +18,13 @@ function parseDevicesResult(result: string): Array { if (!result) { return []; } - + const devices = []; const lines = result.trim().split(/\r?\n/); - + for (let i=0; i < lines.length; i++) { let words = lines[i].split(/[ ,\t]+/).filter((w) => w !== ''); - + if (words[1] === 'device') { devices.push(words[0]); } @@ -42,8 +42,8 @@ function getDevices(): Array { } catch (e) { return []; } - - + + } module.exports = { diff --git a/local-cli/runAndroid/runAndroid.js b/local-cli/runAndroid/runAndroid.js index e2552788b..132a50806 100644 --- a/local-cli/runAndroid/runAndroid.js +++ b/local-cli/runAndroid/runAndroid.js @@ -120,7 +120,7 @@ function buildAndRun(args, reject) { : 'adb'; const devices = adb.getDevices(); - + if (devices && devices.length > 0) { devices.forEach((device) => { @@ -130,8 +130,8 @@ function buildAndRun(args, reject) { `Starting the app on ${device} (${adbPath} ${adbArgs.join(' ')})...` )); - child_process.spawnSync(adbPath, adbArgs, {stdio: 'inherit'}); - }); + child_process.spawnSync(adbPath, adbArgs, {stdio: 'inherit'}); + }); } else { // If we cannot execute based on adb devices output, fall back to // shell am start diff --git a/packager/react-native-xcode.sh b/packager/react-native-xcode.sh index 9e6ffbe43..57201babb 100755 --- a/packager/react-native-xcode.sh +++ b/packager/react-native-xcode.sh @@ -14,11 +14,11 @@ case "$CONFIGURATION" in Debug) # Speed up build times by skipping the creation of the offline package for debug # builds on the simulator since the packager is supposed to be running anyways. - if [[ "$PLATFORM_NAME" = "iphonesimulator" ]]; then - echo "Skipping bundling for Simulator platform" - exit 0; + if [[ "$PLATFORM_NAME" = "iphonesimulator" ]]; then + echo "Skipping bundling for Simulator platform" + exit 0; fi - + DEV=true ;; "") diff --git a/packager/react-packager/src/Resolver/polyfills/String.prototype.es6.js b/packager/react-packager/src/Resolver/polyfills/String.prototype.es6.js index 59e5351d3..52bbc3147 100644 --- a/packager/react-packager/src/Resolver/polyfills/String.prototype.es6.js +++ b/packager/react-packager/src/Resolver/polyfills/String.prototype.es6.js @@ -77,7 +77,7 @@ if (!String.prototype.includes) { if (typeof start !== 'number') { start = 0; } - + if (start + search.length > this.length) { return false; } else { diff --git a/website/src/react-native/index.js b/website/src/react-native/index.js index 04a624eb3..f0f239618 100644 --- a/website/src/react-native/index.js +++ b/website/src/react-native/index.js @@ -48,10 +48,10 @@ var index = React.createClass({ {`// iOS -import React, { +import React, { Component, - TabBarIOS, - NavigatorIOS + TabBarIOS, + NavigatorIOS } from 'react-native'; class App extends Component { @@ -70,11 +70,11 @@ class App extends Component { {`// Android -import React, { +import React, { Component, - DrawerLayoutAndroid, - ProgressBarAndroid, - Text + DrawerLayoutAndroid, + ProgressBarAndroid, + Text } from 'react-native'; class App extends Component { @@ -105,7 +105,7 @@ class App extends Component { {`// iOS & Android -import React, { +import React, { Component, ScrollView, TouchableHighlight, @@ -134,10 +134,10 @@ class TouchDemo extends Component { var React, { Component, - Image, - StyleSheet, - Text, - View + Image, + StyleSheet, + Text, + View } from 'react-native'; class ReactNative extends Component { @@ -176,9 +176,9 @@ var styles = StyleSheet.create({ {`// iOS & Android -import React, { +import React, { Component, - Text + Text } from 'react-native'; class GeoInfo extends Component { @@ -283,7 +283,7 @@ RCT_EXPORT_VIEW_PROPERTY(myCustomProperty, NSString); {`// JavaScript -import React, { +import React, { Component, requireNativeComponent } from 'react-native'; @@ -379,7 +379,7 @@ public class MyCustomViewManager extends SimpleViewManager { import React, { Component, - requireNativeComponent + requireNativeComponent } from 'react-native'; var NativeMyCustomView = requireNativeComponent('MyCustomView', MyCustomView);