Skip to content

Commit

Permalink
fix: Use nativeOrder for ByteBuffer order
Browse files Browse the repository at this point in the history
  • Loading branch information
mrousavy committed Sep 16, 2024
1 parent 05776c2 commit 582eab0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ using namespace facebook;
class ByteBufferArrayBuffer final : public ArrayBuffer {
public:
explicit ByteBufferArrayBuffer(const jni::alias_ref<jni::JByteBuffer>& byteBuffer) : _byteBuffer(jni::make_global(byteBuffer)) {
_byteBuffer->order(jni::JByteOrder::bigEndian());
_byteBuffer->order(jni::JByteOrder::nativeOrder());
}

public:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -78,12 +78,12 @@ class JArrayBuffer final : public jni::HybridClass<JArrayBuffer> {
size_t size = _arrayBuffer->size();
if (copyIfNeeded) {
auto buffer = jni::JByteBuffer::allocateDirect(size);
buffer->order(jni::JByteOrder::bigEndian());
buffer->order(jni::JByteOrder::nativeOrder());
memcpy(buffer->getDirectAddress(), _arrayBuffer->data(), size);
return buffer;
} else {
auto buffer = jni::JByteBuffer::wrapBytes(_arrayBuffer->data(), size);
buffer->order(jni::JByteOrder::bigEndian());
buffer->order(jni::JByteOrder::nativeOrder());
return buffer;
}
}
Expand Down

0 comments on commit 582eab0

Please sign in to comment.