Skip to content

Commit 32a71e6

Browse files
authored
Avoid unnecessary borrowing. (#112)
Found by clippy's needless_borrow check. Signed-off-by: Piotr Sikora <[email protected]>
1 parent 50b17c0 commit 32a71e6

File tree

2 files changed

+21
-21
lines changed

2 files changed

+21
-21
lines changed

src/hostcalls.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1053,7 +1053,7 @@ mod utils {
10531053
}
10541054
let mut bytes: Bytes = Vec::with_capacity(size);
10551055
for part in &path {
1056-
bytes.extend_from_slice(&part.as_bytes());
1056+
bytes.extend_from_slice(part.as_bytes());
10571057
bytes.push(0);
10581058
}
10591059
bytes.pop();
@@ -1072,9 +1072,9 @@ mod utils {
10721072
bytes.extend_from_slice(&value.len().to_le_bytes());
10731073
}
10741074
for (name, value) in &map {
1075-
bytes.extend_from_slice(&name.as_bytes());
1075+
bytes.extend_from_slice(name.as_bytes());
10761076
bytes.push(0);
1077-
bytes.extend_from_slice(&value.as_bytes());
1077+
bytes.extend_from_slice(value.as_bytes());
10781078
bytes.push(0);
10791079
}
10801080
bytes
@@ -1092,9 +1092,9 @@ mod utils {
10921092
bytes.extend_from_slice(&value.len().to_le_bytes());
10931093
}
10941094
for (name, value) in &map {
1095-
bytes.extend_from_slice(&name.as_bytes());
1095+
bytes.extend_from_slice(name.as_bytes());
10961096
bytes.push(0);
1097-
bytes.extend_from_slice(&value);
1097+
bytes.extend_from_slice(value);
10981098
bytes.push(0);
10991099
}
11001100
bytes

src/traits.rs

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ pub trait Context {
8787
}
8888

8989
fn get_http_call_response_header(&self, name: &str) -> Option<String> {
90-
hostcalls::get_map_value(MapType::HttpCallResponseHeaders, &name).unwrap()
90+
hostcalls::get_map_value(MapType::HttpCallResponseHeaders, name).unwrap()
9191
}
9292

9393
fn get_http_call_response_header_bytes(&self, name: &str) -> Option<Bytes> {
@@ -107,7 +107,7 @@ pub trait Context {
107107
}
108108

109109
fn get_http_call_response_trailer(&self, name: &str) -> Option<String> {
110-
hostcalls::get_map_value(MapType::HttpCallResponseTrailers, &name).unwrap()
110+
hostcalls::get_map_value(MapType::HttpCallResponseTrailers, name).unwrap()
111111
}
112112

113113
fn get_http_call_response_trailer_bytes(&self, name: &str) -> Option<Bytes> {
@@ -160,7 +160,7 @@ pub trait Context {
160160
}
161161

162162
fn get_grpc_stream_initial_metadata_value(&self, name: &str) -> Option<Bytes> {
163-
hostcalls::get_map_value_bytes(MapType::GrpcReceiveInitialMetadata, &name).unwrap()
163+
hostcalls::get_map_value_bytes(MapType::GrpcReceiveInitialMetadata, name).unwrap()
164164
}
165165

166166
fn send_grpc_stream_message(&self, token_id: u32, message: Option<&[u8]>, end_stream: bool) {
@@ -180,7 +180,7 @@ pub trait Context {
180180
}
181181

182182
fn get_grpc_stream_trailing_metadata_value(&self, name: &str) -> Option<Bytes> {
183-
hostcalls::get_map_value_bytes(MapType::GrpcReceiveTrailingMetadata, &name).unwrap()
183+
hostcalls::get_map_value_bytes(MapType::GrpcReceiveTrailingMetadata, name).unwrap()
184184
}
185185

186186
fn cancel_grpc_stream(&self, token_id: u32) {
@@ -300,23 +300,23 @@ pub trait HttpContext: Context {
300300
}
301301

302302
fn get_http_request_header(&self, name: &str) -> Option<String> {
303-
hostcalls::get_map_value(MapType::HttpRequestHeaders, &name).unwrap()
303+
hostcalls::get_map_value(MapType::HttpRequestHeaders, name).unwrap()
304304
}
305305

306306
fn get_http_request_header_bytes(&self, name: &str) -> Option<Bytes> {
307307
hostcalls::get_map_value_bytes(MapType::HttpRequestHeaders, name).unwrap()
308308
}
309309

310310
fn set_http_request_header(&self, name: &str, value: Option<&str>) {
311-
hostcalls::set_map_value(MapType::HttpRequestHeaders, &name, value).unwrap()
311+
hostcalls::set_map_value(MapType::HttpRequestHeaders, name, value).unwrap()
312312
}
313313

314314
fn set_http_request_header_bytes(&self, name: &str, value: Option<&[u8]>) {
315315
hostcalls::set_map_value_bytes(MapType::HttpRequestHeaders, name, value).unwrap()
316316
}
317317

318318
fn add_http_request_header(&self, name: &str, value: &str) {
319-
hostcalls::add_map_value(MapType::HttpRequestHeaders, &name, value).unwrap()
319+
hostcalls::add_map_value(MapType::HttpRequestHeaders, name, value).unwrap()
320320
}
321321

322322
fn add_http_request_header_bytes(&self, name: &str, value: &[u8]) {
@@ -356,23 +356,23 @@ pub trait HttpContext: Context {
356356
}
357357

358358
fn get_http_request_trailer(&self, name: &str) -> Option<String> {
359-
hostcalls::get_map_value(MapType::HttpRequestTrailers, &name).unwrap()
359+
hostcalls::get_map_value(MapType::HttpRequestTrailers, name).unwrap()
360360
}
361361

362362
fn get_http_request_trailer_bytes(&self, name: &str) -> Option<Bytes> {
363363
hostcalls::get_map_value_bytes(MapType::HttpRequestTrailers, name).unwrap()
364364
}
365365

366366
fn set_http_request_trailer(&self, name: &str, value: Option<&str>) {
367-
hostcalls::set_map_value(MapType::HttpRequestTrailers, &name, value).unwrap()
367+
hostcalls::set_map_value(MapType::HttpRequestTrailers, name, value).unwrap()
368368
}
369369

370370
fn set_http_request_trailer_bytes(&self, name: &str, value: Option<&[u8]>) {
371371
hostcalls::set_map_value_bytes(MapType::HttpRequestTrailers, name, value).unwrap()
372372
}
373373

374374
fn add_http_request_trailer(&self, name: &str, value: &str) {
375-
hostcalls::add_map_value(MapType::HttpRequestTrailers, &name, value).unwrap()
375+
hostcalls::add_map_value(MapType::HttpRequestTrailers, name, value).unwrap()
376376
}
377377

378378
fn add_http_request_trailer_bytes(&self, name: &str, value: &[u8]) {
@@ -404,23 +404,23 @@ pub trait HttpContext: Context {
404404
}
405405

406406
fn get_http_response_header(&self, name: &str) -> Option<String> {
407-
hostcalls::get_map_value(MapType::HttpResponseHeaders, &name).unwrap()
407+
hostcalls::get_map_value(MapType::HttpResponseHeaders, name).unwrap()
408408
}
409409

410410
fn get_http_response_header_bytes(&self, name: &str) -> Option<Bytes> {
411411
hostcalls::get_map_value_bytes(MapType::HttpResponseHeaders, name).unwrap()
412412
}
413413

414414
fn set_http_response_header(&self, name: &str, value: Option<&str>) {
415-
hostcalls::set_map_value(MapType::HttpResponseHeaders, &name, value).unwrap()
415+
hostcalls::set_map_value(MapType::HttpResponseHeaders, name, value).unwrap()
416416
}
417417

418418
fn set_http_response_header_bytes(&self, name: &str, value: Option<&[u8]>) {
419419
hostcalls::set_map_value_bytes(MapType::HttpResponseHeaders, name, value).unwrap()
420420
}
421421

422422
fn add_http_response_header(&self, name: &str, value: &str) {
423-
hostcalls::add_map_value(MapType::HttpResponseHeaders, &name, value).unwrap()
423+
hostcalls::add_map_value(MapType::HttpResponseHeaders, name, value).unwrap()
424424
}
425425

426426
fn add_http_response_header_bytes(&self, name: &str, value: &[u8]) {
@@ -460,23 +460,23 @@ pub trait HttpContext: Context {
460460
}
461461

462462
fn get_http_response_trailer(&self, name: &str) -> Option<String> {
463-
hostcalls::get_map_value(MapType::HttpResponseTrailers, &name).unwrap()
463+
hostcalls::get_map_value(MapType::HttpResponseTrailers, name).unwrap()
464464
}
465465

466466
fn get_http_response_trailer_bytes(&self, name: &str) -> Option<Bytes> {
467467
hostcalls::get_map_value_bytes(MapType::HttpResponseTrailers, name).unwrap()
468468
}
469469

470470
fn set_http_response_trailer(&self, name: &str, value: Option<&str>) {
471-
hostcalls::set_map_value(MapType::HttpResponseTrailers, &name, value).unwrap()
471+
hostcalls::set_map_value(MapType::HttpResponseTrailers, name, value).unwrap()
472472
}
473473

474474
fn set_http_response_trailer_bytes(&self, name: &str, value: Option<&[u8]>) {
475475
hostcalls::set_map_value_bytes(MapType::HttpResponseTrailers, name, value).unwrap()
476476
}
477477

478478
fn add_http_response_trailer(&self, name: &str, value: &str) {
479-
hostcalls::add_map_value(MapType::HttpResponseTrailers, &name, value).unwrap()
479+
hostcalls::add_map_value(MapType::HttpResponseTrailers, name, value).unwrap()
480480
}
481481

482482
fn add_http_response_trailer_bytes(&self, name: &str, value: &[u8]) {

0 commit comments

Comments
 (0)