Skip to content

Commit 89eb287

Browse files
passyrickhanlonii
authored andcommitted
clang-format cpp
Summary: Ran `arc f` against some CPP files to be modified with the next diff to reduce the churn on it. ## Changelog [Android] [Changed] - Formatted cpp/h code with clang-format Reviewed By: javache Differential Revision: D19371785 fbshipit-source-id: b7f7b92c4cb9ec7f8da728bb577db29cf11fbb39
1 parent 3c3b4cf commit 89eb287

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

44 files changed

+924
-658
lines changed

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/Binding.cpp

Lines changed: 134 additions & 87 deletions
Large diffs are not rendered by default.

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/Binding.h

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -76,17 +76,17 @@ class Binding : public jni::HybridClass<Binding>, public SchedulerDelegate {
7676
const ShadowView &shadowView);
7777

7878
void schedulerDidDispatchCommand(
79-
const ShadowView &shadowView,
80-
std::string const &commandName,
81-
folly::dynamic const args);
79+
const ShadowView &shadowView,
80+
std::string const &commandName,
81+
folly::dynamic const args);
8282

8383
void setPixelDensity(float pointScaleFactor);
8484

8585
void schedulerDidSetJSResponder(
86-
SurfaceId surfaceId,
87-
const ShadowView &shadowView,
88-
const ShadowView &initialShadowView,
89-
bool blockNativeResponder);
86+
SurfaceId surfaceId,
87+
const ShadowView &shadowView,
88+
const ShadowView &initialShadowView,
89+
bool blockNativeResponder);
9090

9191
void schedulerDidClearJSResponder();
9292

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/ComponentFactoryDelegate.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
#include <fb/fbjni.h>
1111
#include <jsi/jsi.h>
1212
#include <react/uimanager/ComponentDescriptorRegistry.h>
13-
#include <react/utils/ContextContainer.h>
1413
#include <react/uimanager/Scheduler.h>
14+
#include <react/utils/ContextContainer.h>
1515
#include <mutex>
1616
#include <unordered_set>
1717

@@ -25,7 +25,7 @@ class Instance;
2525
class ComponentFactoryDelegate
2626
: public jni::HybridClass<ComponentFactoryDelegate> {
2727
public:
28-
constexpr static const char* const kJavaDescriptor =
28+
constexpr static const char *const kJavaDescriptor =
2929
"Lcom/facebook/react/fabric/ComponentFactoryDelegate;";
3030

3131
static void registerNatives();

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/EventBeatManager.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ namespace facebook {
1313
namespace react {
1414

1515
EventBeatManager::EventBeatManager(
16-
jni::alias_ref<EventBeatManager::jhybriddata> jhybridobject)
17-
: jhybridobject_(jhybridobject) {}
16+
jni::alias_ref<EventBeatManager::jhybriddata> jhybridobject)
17+
: jhybridobject_(jhybridobject) {}
1818

1919
jni::local_ref<EventBeatManager::jhybriddata> EventBeatManager::initHybrid(
2020
jni::alias_ref<EventBeatManager::jhybriddata> jhybridobject) {

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/EventEmitterWrapper.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ EventEmitterWrapper::initHybrid(jni::alias_ref<jclass>) {
2020

2121
void EventEmitterWrapper::invokeEvent(
2222
std::string eventName,
23-
NativeMap* payload) {
23+
NativeMap *payload) {
2424
eventEmitter->dispatchEvent(
2525
eventName, payload->consume(), EventPriority::AsynchronousBatched);
2626
}
@@ -33,4 +33,4 @@ void EventEmitterWrapper::registerNatives() {
3333
}
3434

3535
} // namespace react
36-
} // namespace facebook
36+
} // namespace facebook

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/EventEmitterWrapper.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@ class Instance;
1818

1919
class EventEmitterWrapper : public jni::HybridClass<EventEmitterWrapper> {
2020
public:
21-
constexpr static const char* const kJavaDescriptor =
21+
constexpr static const char *const kJavaDescriptor =
2222
"Lcom/facebook/react/fabric/events/EventEmitterWrapper;";
2323

2424
static void registerNatives();
2525

2626
SharedEventEmitter eventEmitter;
2727

28-
void invokeEvent(std::string eventName, NativeMap* params);
28+
void invokeEvent(std::string eventName, NativeMap *params);
2929

3030
private:
3131
static jni::local_ref<jhybriddata> initHybrid(jni::alias_ref<jclass>);

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/NodeStateWrapper.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ using namespace facebook::jni;
1414
namespace facebook {
1515
namespace react {
1616

17-
jni::local_ref<NodeStateWrapper::jhybriddata>
18-
NodeStateWrapper::initHybrid(jni::alias_ref<jclass>) {
17+
jni::local_ref<NodeStateWrapper::jhybriddata> NodeStateWrapper::initHybrid(
18+
jni::alias_ref<jclass>) {
1919
return makeCxxInstance();
2020
}
2121

@@ -26,7 +26,7 @@ jni::local_ref<ReadableNativeMap::jhybridobject> NodeStateWrapper::getState() {
2626
return readableNativeMap;
2727
}
2828

29-
void NodeStateWrapper::updateState(ReadableNativeMap* map) {
29+
void NodeStateWrapper::updateState(ReadableNativeMap *map) {
3030
// Get folly::dynamic from map
3131
auto dynamicMap = map->consume();
3232
// Set state

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/NodeStateWrapper.h

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,18 @@ namespace react {
1616

1717
class NodeStateWrapper : public jni::HybridClass<NodeStateWrapper> {
1818
public:
19-
constexpr static const char* const kJavaDescriptor =
19+
constexpr static const char *const kJavaDescriptor =
2020
"Lcom/facebook/react/fabric/NodeStateWrapper;";
2121

2222
NodeStateWrapper() {}
2323

2424
static void registerNatives();
2525

2626
jni::local_ref<ReadableNativeMap::jhybridobject> getState();
27-
void updateState(ReadableNativeMap* map);
27+
void updateState(ReadableNativeMap *map);
28+
29+
const State *state_;
2830

29-
const State* state_;
3031
private:
3132
static jni::local_ref<jhybriddata> initHybrid(jni::alias_ref<jclass>);
3233
};

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/OnLoad.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
#include "EventEmitterWrapper.h"
1414
#include "StateWrapperImpl.h"
1515

16-
JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM* vm, void*) {
16+
JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM *vm, void *) {
1717
return facebook::xplat::initialize(vm, [] {
1818
facebook::react::Binding::registerNatives();
1919
facebook::react::EventBeatManager::registerNatives();

ReactAndroid/src/main/java/com/facebook/react/fabric/jni/ReactNativeConfigHolder.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,29 +11,29 @@
1111

1212
using namespace facebook::react;
1313

14-
bool ReactNativeConfigHolder::getBool(const std::string& param) const {
14+
bool ReactNativeConfigHolder::getBool(const std::string &param) const {
1515
static const auto method = facebook::jni::findClassStatic(
1616
"com/facebook/react/fabric/ReactNativeConfig")
1717
->getMethod<jboolean(jstring)>("getBool");
1818
return method(reactNativeConfig_, facebook::jni::make_jstring(param).get());
1919
}
2020

21-
std::string ReactNativeConfigHolder::getString(const std::string& param) const {
21+
std::string ReactNativeConfigHolder::getString(const std::string &param) const {
2222
static const auto method = facebook::jni::findClassStatic(
2323
"com/facebook/react/fabric/ReactNativeConfig")
2424
->getMethod<jstring(jstring)>("getString");
2525
return method(reactNativeConfig_, facebook::jni::make_jstring(param).get())
2626
->toString();
2727
}
2828

29-
int64_t ReactNativeConfigHolder::getInt64(const std::string& param) const {
29+
int64_t ReactNativeConfigHolder::getInt64(const std::string &param) const {
3030
static const auto method = facebook::jni::findClassStatic(
3131
"com/facebook/react/fabric/ReactNativeConfig")
3232
->getMethod<jint(jstring)>("getInt64");
3333
return method(reactNativeConfig_, facebook::jni::make_jstring(param).get());
3434
}
3535

36-
double ReactNativeConfigHolder::getDouble(const std::string& param) const {
36+
double ReactNativeConfigHolder::getDouble(const std::string &param) const {
3737
static const auto method = facebook::jni::findClassStatic(
3838
"com/facebook/react/fabric/ReactNativeConfig")
3939
->getMethod<jdouble(jstring)>("getDouble");

0 commit comments

Comments
 (0)