Skip to content

Commit 509da6b

Browse files
committed
refactor(lua): use enumerate() instead of manual counter
Replace manual counter increment with Iterator::enumerate() for cleaner iteration over URI args, post args, and headers.
1 parent b87c838 commit 509da6b

1 file changed

Lines changed: 3 additions & 9 deletions

File tree

src/http/lua.rs

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -406,13 +406,11 @@ impl UserData for CandyReq {
406406

407407
let limit = max_args.unwrap_or(100);
408408
let table = lua.create_table()?;
409-
let mut count = 0;
410409

411-
for (k, v) in &state.uri_args.0 {
410+
for (count, (k, v)) in state.uri_args.0.iter().enumerate() {
412411
if limit > 0 && count >= limit {
413412
break;
414413
}
415-
count += 1;
416414

417415
if table.contains_key(k.clone())? {
418416
let existing: mlua::Value = table.get(k.clone())?;
@@ -555,13 +553,11 @@ impl UserData for CandyReq {
555553
let post_args = state.post_args.as_ref().unwrap();
556554
let limit = max_args.unwrap_or(100);
557555
let table = lua.create_table()?;
558-
let mut count = 0;
559556

560-
for (k, v) in &post_args.0 {
557+
for (count, (k, v)) in post_args.0.iter().enumerate() {
561558
if limit > 0 && count >= limit {
562559
break;
563560
}
564-
count += 1;
565561

566562
if table.contains_key(k.clone())? {
567563
let existing: mlua::Value = table.get(k.clone())?;
@@ -610,13 +606,11 @@ impl UserData for CandyReq {
610606
let limit = max_headers.unwrap_or(100);
611607
let preserve_case = raw.unwrap_or(false);
612608
let table = lua.create_table()?;
613-
let mut count = 0;
614609

615-
for (name, value) in headers.iter() {
610+
for (count, (name, value)) in headers.iter().enumerate() {
616611
if limit > 0 && count >= limit {
617612
break;
618613
}
619-
count += 1;
620614

621615
let key = if preserve_case {
622616
name.as_str().to_string()

0 commit comments

Comments
 (0)