Skip to content

Commit d1685c7

Browse files
rohitjain85microsoft-github-bot[bot]
authored andcommitted
Moving exception flag to LOCAL_CFLAGS to reduce the diff (#24)
1 parent 2944ea3 commit d1685c7

File tree

5 files changed

+12
-10
lines changed

5 files changed

+12
-10
lines changed

ReactAndroid/src/main/jni/first-party/fb/Android.mk

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ LOCAL_SRC_FILES:= \
2929
LOCAL_C_INCLUDES := $(LOCAL_PATH)/include
3030
LOCAL_EXPORT_C_INCLUDES := $(LOCAL_PATH)/include
3131

32-
LOCAL_CFLAGS := -DLOG_TAG=\"libfb\" -DDISABLE_CPUCAP -DDISABLE_XPLAT
33-
LOCAL_CXXFLAGS += -frtti -fexceptions
32+
LOCAL_CFLAGS := -DLOG_TAG=\"libfb\" -DDISABLE_CPUCAP -DDISABLE_XPLAT -fexceptions -frtti
33+
3434
# include/utils/threads.h has unused parameters
3535
LOCAL_CFLAGS += -Wno-unused-parameter
3636
ifeq ($(TOOLCHAIN_PERMISSIVE),true)

ReactAndroid/src/main/jni/first-party/fbgloginit/Android.mk

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ LOCAL_SRC_FILES:= \
1212
LOCAL_C_INCLUDES := $(LOCAL_PATH)
1313
LOCAL_EXPORT_C_INCLUDES := $(LOCAL_PATH)
1414

15-
LOCAL_CFLAGS := -fno-omit-frame-pointer
16-
LOCAL_CXXFLAGS := -fexceptions
15+
LOCAL_CFLAGS := -fexceptions -fno-omit-frame-pointer
1716

1817
LOCAL_LDLIBS := -llog
1918

ReactAndroid/src/main/jni/first-party/yogajni/Android.mk

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,7 @@ LOCAL_SRC_FILES := \
1414

1515
LOCAL_C_INCLUDES := $(LOCAL_PATH)/jni
1616

17-
LOCAL_CFLAGS += -fvisibility=hidden -O3
18-
LOCAL_CXXFLAGS += -frtti -fexceptions
17+
LOCAL_CFLAGS += -fvisibility=hidden -frtti -fexceptions -O3
1918

2019
LOCAL_LDLIBS += -landroid -llog
2120
LOCAL_STATIC_LIBRARIES := libyogacore

ReactAndroid/src/main/jni/react/jni/Android.mk

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@ include $(CLEAR_VARS)
1010
# Flag to enable V8 in react-native code
1111
V8_ENABLED := 1
1212

13+
# Name of this module.
14+
#
15+
# Other modules can depend on this one by adding libreactnativejni to their
16+
# LOCAL_SHARED_LIBRARIES variable.
1317
LOCAL_MODULE := reactnativejni
1418

1519
LOCAL_SRC_FILES := \
@@ -35,6 +39,7 @@ LOCAL_SRC_FILES := \
3539
WritableNativeArray.cpp \
3640
WritableNativeMap.cpp \
3741

42+
# Include . in the header search path for all source files in this module.
3843
LOCAL_C_INCLUDES := $(LOCAL_PATH)
3944

4045
# Include ./../../ in the header search path for modules that depend on
@@ -44,8 +49,7 @@ LOCAL_C_INCLUDES := $(LOCAL_PATH)
4449
# ./../ == react
4550
LOCAL_EXPORT_C_INCLUDES := $(LOCAL_PATH)/../..
4651

47-
LOCAL_CFLAGS += -fvisibility=hidden
48-
LOCAL_CFLAGS += -fexceptions -frtti
52+
LOCAL_CFLAGS += -fvisibility=hidden -fexceptions -frtti
4953

5054
LOCAL_LDLIBS += -landroid
5155

@@ -68,7 +72,6 @@ else
6872
LOCAL_SHARED_LIBRARIES += libjsc
6973
endif
7074

71-
7275
# The static libraries (.a files) that this module depends on.
7376
LOCAL_STATIC_LIBRARIES := libreactnative
7477

ReactCommon/cxxreact/Android.mk

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,10 @@ LOCAL_EXPORT_C_INCLUDES := $(LOCAL_C_INCLUDES)
3434
LOCAL_CFLAGS := \
3535
-DLOG_TAG=\"ReactNative\"
3636

37+
LOCAL_CFLAGS += -fexceptions -frtti
38+
3739
LOCAL_STATIC_LIBRARIES := boost
3840
LOCAL_SHARED_LIBRARIES := libfb libfolly_json libglog
39-
LOCAL_CXXFLAGS += -fexceptions -frtti
4041

4142
LOCAL_V8_FILES := \
4243
File.cpp \

0 commit comments

Comments
 (0)