Skip to content

Fix clippy warnings & update test#73

Merged
taiki-e merged 4 commits intomainfrom
taiki-e/clippy
Feb 27, 2025
Merged

Fix clippy warnings & update test#73
taiki-e merged 4 commits intomainfrom
taiki-e/clippy

Conversation

@taiki-e
Copy link
Contributor

@taiki-e taiki-e commented Feb 27, 2025

No description provided.

```
warning: unnecessary semicolon
   --> src/collada/mod.rs:289:10
    |
289 |         };
    |          ^ help: remove
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_semicolon
    = note: `-W clippy::unnecessary-semicolon` implied by `-W clippy::pedantic`
    = help: to override `-W clippy::pedantic` add `#[allow(clippy::unnecessary_semicolon)]`

warning: unnecessary semicolon
   --> src/obj/mod.rs:372:6
    |
372 |     };
    |      ^ help: remove
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_semicolon

warning: unnecessary semicolon
   --> src/obj/mod.rs:396:10
    |
396 |         };
    |          ^ help: remove
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_semicolon

warning: unnecessary semicolon
   --> src/obj/mod.rs:421:10
    |
421 |         };
    |          ^ help: remove
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_semicolon
```
```
warning: this `continue` expression is redundant
   --> src/utils/float/decimal.rs:355:13
    |
355 |             continue;
    |             ^^^^^^^^
    |
    = help: consider dropping the `continue` expression
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_continue
    = note: `-W clippy::needless-continue` implied by `-W clippy::pedantic`
    = help: to override `-W clippy::pedantic` add `#[allow(clippy::needless_continue)]`

warning: this `continue` expression is redundant
    --> src/obj/mod.rs:1335:9
     |
1335 |         continue;
     |         ^^^^^^^^
     |
     = help: consider dropping the `continue` expression
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_continue
```
```
warning: calling `as_bytes` after slicing a string
   --> src/utils/xml.rs:259:37
    |
259 |         start += memchr_naive(b'>', self.document().input_text()[start..].as_bytes())
    |                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `&self.document().input_text().as_bytes()[start..]`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#sliced_string_as_bytes
    = note: `-W clippy::sliced-string-as-bytes` implied by `-W clippy::all`
    = help: to override `-W clippy::all` add `#[allow(clippy::sliced_string_as_bytes)]`
```
@taiki-e taiki-e closed this Feb 27, 2025
@taiki-e taiki-e reopened this Feb 27, 2025
@taiki-e taiki-e changed the title Fix clippy warnings Fix clippy warnings & update test Feb 27, 2025
@taiki-e taiki-e merged commit 51ccbd8 into main Feb 27, 2025
7 checks passed
@taiki-e taiki-e deleted the taiki-e/clippy branch February 27, 2025 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant