Skip to content

Commit 56b1ea1

Browse files
committed
cargo fmt
1 parent 376cdbe commit 56b1ea1

8 files changed

Lines changed: 49 additions & 39 deletions

File tree

zenoh-jni/build.rs

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,27 @@ fn main() {
1414
.string_decoder("crate::utils::decode_string")
1515
.byte_array_decoder("crate::utils::decode_byte_array")
1616
.encoding_decoder("crate::utils::decode_encoding")
17-
.enum_decoder("CongestionControl", "crate::utils::decode_congestion_control")
17+
.enum_decoder(
18+
"CongestionControl",
19+
"crate::utils::decode_congestion_control",
20+
)
1821
.enum_decoder("Priority", "crate::utils::decode_priority")
1922
.enum_decoder("Reliability", "crate::utils::decode_reliability")
2023
.enum_decoder("QueryTarget", "crate::utils::decode_query_target")
2124
.enum_decoder("ConsolidationMode", "crate::utils::decode_consolidation")
2225
.enum_decoder("ReplyKeyExpr", "crate::utils::decode_reply_key_expr")
23-
.callback_decoder("Sample", "crate::sample_callback::process_kotlin_sample_callback")
24-
.callback_decoder("Query", "crate::sample_callback::process_kotlin_query_callback")
25-
.callback_decoder("Reply", "crate::sample_callback::process_kotlin_reply_callback")
26+
.callback_decoder(
27+
"Sample",
28+
"crate::sample_callback::process_kotlin_sample_callback",
29+
)
30+
.callback_decoder(
31+
"Query",
32+
"crate::sample_callback::process_kotlin_query_callback",
33+
)
34+
.callback_decoder(
35+
"Reply",
36+
"crate::sample_callback::process_kotlin_reply_callback",
37+
)
2638
.consume_arg("close_session", "session")
2739
.consume_arg("undeclare_key_expr", "key_expr")
2840
.return_wrapper(

zenoh-jni/src/config.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ use jni::{
2121
};
2222
use zenoh::Config;
2323

24-
use crate::owned_object::OwnedObject;
2524
use crate::errors::ZResult;
25+
use crate::owned_object::OwnedObject;
2626
use crate::{throw_exception, utils::decode_string};
2727

2828
/// Loads the default configuration, returning a raw pointer to it.

zenoh-jni/src/errors.rs

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,11 @@ use jni::JNIEnv;
1717
#[macro_export]
1818
macro_rules! throw_exception {
1919
($env:expr, $err:expr) => {{
20-
let _ = <crate::errors::ZError as crate::errors::ThrowOnJvm>::throw_on_jvm(
21-
&$err,
22-
&mut $env,
23-
)
24-
.map_err(|err| {
25-
tracing::error!("Unable to throw exception: {}", err);
26-
});
20+
let _ =
21+
<crate::errors::ZError as crate::errors::ThrowOnJvm>::throw_on_jvm(&$err, &mut $env)
22+
.map_err(|err| {
23+
tracing::error!("Unable to throw exception: {}", err);
24+
});
2725
}};
2826
}
2927

zenoh-jni/src/ext/advanced_subscriber.rs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,9 @@ pub unsafe extern "C" fn Java_io_zenoh_jni_JNIAdvancedSubscriber_declareDetectPu
141141
let detect_publishers_subscriber = advanced_subscriber
142142
.detect_publishers()
143143
.history(history != 0)
144-
.callback(process_kotlin_sample_callback(&mut env, callback, on_close)?)
144+
.callback(process_kotlin_sample_callback(
145+
&mut env, callback, on_close,
146+
)?)
145147
.wait()
146148
.map_err(|err| zerror!("Unable to declare detect publishers subscriber: {}", err))?;
147149

@@ -197,7 +199,9 @@ pub unsafe extern "C" fn Java_io_zenoh_jni_JNIAdvancedSubscriber_declareBackgrou
197199
advanced_subscriber
198200
.detect_publishers()
199201
.history(history != 0)
200-
.callback(process_kotlin_sample_callback(&mut env, callback, on_close)?)
202+
.callback(process_kotlin_sample_callback(
203+
&mut env, callback, on_close,
204+
)?)
201205
.background()
202206
.wait()
203207
.map_err(|err| {

zenoh-jni/src/key_expr.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ use zenoh::key_expr::KeyExpr;
2121

2222
use crate::errors::ZResult;
2323
use crate::owned_object::OwnedObject;
24-
use crate::utils::decode_string;
2524
use crate::throw_exception;
25+
use crate::utils::decode_string;
2626

2727
/// Validates the provided `key_expr` to be a valid key expression, returning it back
2828
/// in case of success or throwing an exception in case of failure.

zenoh-jni/src/liveliness.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,9 @@ pub unsafe extern "C" fn Java_io_zenoh_jni_JNISession_declareLivelinessSubscribe
154154
.liveliness()
155155
.declare_subscriber(key_expr.to_owned())
156156
.history(history != 0)
157-
.callback(process_kotlin_sample_callback(&mut env, callback, on_close)?)
157+
.callback(process_kotlin_sample_callback(
158+
&mut env, callback, on_close,
159+
)?)
158160
.wait()
159161
.map_err(|err| zerror!("Unable to declare liveliness subscriber: {}", err))?;
160162

zenoh-jni/src/sample_callback.rs

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,11 @@ use jni::{
1919
sys::{jint, jlong},
2020
JNIEnv,
2121
};
22-
use zenoh::{query::{Query, Reply, ReplyError, ReplyKeyExpr}, sample::Sample, session::EntityGlobalId};
22+
use zenoh::{
23+
query::{Query, Reply, ReplyError, ReplyKeyExpr},
24+
sample::Sample,
25+
session::EntityGlobalId,
26+
};
2327

2428
use crate::{errors::ZResult, utils::*};
2529

@@ -183,11 +187,11 @@ pub(crate) unsafe fn process_kotlin_sample_callback(
183187
Ok(move |sample: Sample| {
184188
on_close.noop();
185189
let _ = || -> ZResult<()> {
186-
let mut env = java_vm.attach_current_thread_as_daemon().map_err(|err| {
187-
zerror!("Unable to attach thread for sample callback: {}", err)
188-
})?;
189-
let byte_array = bytes_to_java_array(&env, sample.payload())
190-
.map(|array| env.auto_local(array))?;
190+
let mut env = java_vm
191+
.attach_current_thread_as_daemon()
192+
.map_err(|err| zerror!("Unable to attach thread for sample callback: {}", err))?;
193+
let byte_array =
194+
bytes_to_java_array(&env, sample.payload()).map(|array| env.auto_local(array))?;
191195

192196
let encoding_id: jint = sample.encoding().id() as jint;
193197
let encoding_schema = match sample.encoding().schema() {
@@ -257,9 +261,9 @@ pub(crate) unsafe fn process_kotlin_reply_callback(
257261
|| -> ZResult<()> {
258262
on_close.noop();
259263
tracing::debug!("Receiving reply through JNI: {:?}", reply);
260-
let mut env = java_vm
261-
.attach_current_thread_as_daemon()
262-
.map_err(|err| zerror!("Unable to attach thread for GET query callback: {}", err))?;
264+
let mut env = java_vm.attach_current_thread_as_daemon().map_err(|err| {
265+
zerror!("Unable to attach thread for GET query callback: {}", err)
266+
})?;
263267
match reply.result() {
264268
Ok(sample) => crate::sample_callback::on_reply_success(
265269
&mut env,
@@ -390,5 +394,3 @@ fn on_query(mut env: JNIEnv, query: Query, callback_global_ref: &GlobalRef) -> Z
390394
});
391395
result
392396
}
393-
394-

zenoh-jni/src/session.rs

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -39,20 +39,10 @@ use zenoh_ext::{
3939
RepliesConfig,
4040
};
4141

42-
use crate::{
43-
errors::ZResult, key_expr::process_kotlin_key_expr, throw_exception, utils::*,
44-
};
42+
use crate::{errors::ZResult, key_expr::process_kotlin_key_expr, throw_exception, utils::*};
4543

4644
include!(concat!(env!("OUT_DIR"), "/zenoh_flat_jni.rs"));
4745

48-
49-
50-
51-
52-
53-
54-
55-
5646
/// Declare an advanced Zenoh subscriber via JNI.
5747
///
5848
/// # Parameters:
@@ -108,7 +98,9 @@ pub unsafe extern "C" fn Java_io_zenoh_jni_JNISession_declareAdvancedSubscriberV
10898
tracing::debug!("Declaring advanced subscriber on '{}'...", key_expr);
10999
let mut builder = session
110100
.declare_subscriber(key_expr.to_owned())
111-
.callback(process_kotlin_sample_callback(&mut env, callback, on_close)?)
101+
.callback(process_kotlin_sample_callback(
102+
&mut env, callback, on_close,
103+
)?)
112104
.advanced();
113105
tracing::debug!("Advanced subscriber declared on '{}'.", key_expr);
114106

0 commit comments

Comments
 (0)