f00116c3a6
Summary: In D17439957, I noted that YogaLogger#log throws a NoMethodFoundException when called from C++ b/c C++ and Java's signatures of that method don't match. C++ uses YogaNodeJNIBase for the first param, Java uses YogaNode. Both my attempts to fix this failed. Attempt #1 - Make Java use YogaNodeJNIBase. This doesn't work because the :java-interface target includes YogaLogger but not YogaNodeJNIBase. Moving YogaLogger to the impl target doesn't work either b/c other files in :java-interface reference YogaLogger. Attempt #2 - Make C++ use YogaNode. This doesn't work b/c we try to call the log method with objects of fbjni type YogaNodeJNIBase. This would be fine in Java since YogaNodeJNIBase extends YogaNode. But fbjni's typing isn't advanced enough to know this, so the Yoga C++ fails to compile. At this point, I was wondering what the value of having this param in the log function at all was. None of the implementations in our codebase use it today. It might be easier to just remove it all together. This also removes a bug with YGNodePrint where we pass a null layout context that eventually causes a SIG_ABRT when we use it to try to find a YogaNode to pass to this function. (https://fburl.com/diffusion/ssw9h8lv). Reviewed By: amir-shalem Differential Revision: D17470379 fbshipit-source-id: 8fc2d95505971a52af2399a9fbb60b63f27f0ec2
43 lines
1.3 KiB
C++
43 lines
1.3 KiB
C++
/*
|
|
* Copyright (c) Facebook, Inc. and its affiliates.
|
|
*
|
|
* This source code is licensed under the MIT license found in the LICENSE
|
|
* file in the root directory of this source tree.
|
|
*/
|
|
#include "YGJTypes.h"
|
|
|
|
using facebook::jni::alias_ref;
|
|
using facebook::jni::local_ref;
|
|
|
|
jfloat JYogaNode::baseline(jfloat width, jfloat height) {
|
|
static auto javaMethod =
|
|
javaClassLocal()->getMethod<jfloat(jfloat, jfloat)>("baseline");
|
|
return javaMethod(self(), width, height);
|
|
}
|
|
|
|
jlong JYogaNode::measure(
|
|
jfloat width,
|
|
jint widthMode,
|
|
jfloat height,
|
|
jint heightMode) {
|
|
static auto javaMethod =
|
|
javaClassLocal()->getMethod<jlong(jfloat, jint, jfloat, jint)>("measure");
|
|
return javaMethod(self(), width, widthMode, height, heightMode);
|
|
}
|
|
|
|
facebook::jni::local_ref<JYogaLogLevel> JYogaLogLevel::fromInt(jint logLevel) {
|
|
static auto javaMethod =
|
|
javaClassStatic()->getStaticMethod<alias_ref<JYogaLogLevel>(jint)>(
|
|
"fromInt");
|
|
return javaMethod(javaClassStatic(), logLevel);
|
|
}
|
|
|
|
void JYogaLogger::log(
|
|
facebook::jni::alias_ref<JYogaLogLevel> logLevel,
|
|
jstring message) {
|
|
static auto javaMethod =
|
|
javaClassLocal()->getMethod<void(alias_ref<JYogaLogLevel>, jstring)>(
|
|
"log");
|
|
javaMethod(self(), logLevel, message);
|
|
}
|