Skip to content
Open
Changes from 1 commit
Commits
Show all changes
59 commits
Select commit Hold shift + click to select a range
daa1e8d
Add Nix flake
ada4a Sep 13, 2025
f82b3c2
parser: bump edition to 2024
ada4a Sep 13, 2025
deb5439
unused/elidable lifetimes
ada4a Sep 14, 2025
5b2c130
redundant refs/derefs
ada4a Sep 14, 2025
8a7c9f7
clippy::uninlined_format_args
ada4a Sep 14, 2025
b094757
clippy::if_not_else
ada4a Sep 14, 2025
659d85e
clippy::needless_return
ada4a Sep 14, 2025
8df056e
explicit iterator loops
ada4a Sep 14, 2025
52d5270
clippy::useless_conversion
ada4a Sep 16, 2025
258e3b7
simplify thanks to the previous commit
ada4a Sep 16, 2025
60602d0
simplify `match`es
ada4a Sep 13, 2025
88731ed
use `std::iter::zip`
ada4a Sep 15, 2025
82259d8
clippy::inefficient_to_string
ada4a Sep 15, 2025
ef55ea7
clippy::ignored_unit_patterns
ada4a Sep 15, 2025
69cb114
derive `Default` for `IssueLinksCheckCommitsConfig`
ada4a Sep 15, 2025
8760f36
use more idiomatic iterator/`Option` adaptors
ada4a Sep 15, 2025
32ea017
clean-up consts
ada4a Sep 15, 2025
778ba02
use `let-else`
ada4a Sep 15, 2025
031aa5d
clippy::cast_lossless
ada4a Sep 15, 2025
554289e
documentation lints
ada4a Sep 15, 2025
d978fc5
reborrow instead of slicing
ada4a Sep 15, 2025
d15c931
add/remove semicolons
ada4a Sep 15, 2025
b331922
clippy::write_with_newline
ada4a Sep 15, 2025
38ba2dd
clippy::manual_string_new
ada4a Sep 15, 2025
c27e100
clippy::single_char_pattern
ada4a Sep 15, 2025
9989a85
clippy::implicit_clone
ada4a Sep 15, 2025
93a80e0
get `command` without unwrapping
ada4a Sep 15, 2025
1da5e71
`write!` _inside_ `match`
ada4a Sep 15, 2025
e2903b5
clippy::unneded_struct_pattern
ada4a Sep 15, 2025
01d4d27
clippy::manual_strip
ada4a Sep 15, 2025
867b95f
clippy::manual_assert
ada4a Sep 15, 2025
d92c33d
needless `into_iter`
ada4a Sep 14, 2025
7d729ff
clippy::legacy_numeric_constants
ada4a Sep 14, 2025
8372962
clippy::inconsistent_struct_constructor
ada4a Sep 14, 2025
efdd2a8
clippy::redundant_closure_for_method_calls
ada4a Sep 14, 2025
f846986
clippy::manual_is_variant_and
ada4a Sep 14, 2025
6ba8cc3
clippy::needless_raw_string_hashes
ada4a Sep 14, 2025
5ed9adf
clippy::question_mark
ada4a Sep 14, 2025
41bde06
construct `HashMap` from array
ada4a Sep 14, 2025
bc1d4ab
clippy::collapsible{,_else}_if
ada4a Sep 14, 2025
ea0801a
use `Result::ok_or` for trying out multiple things
ada4a Sep 14, 2025
05ecbd4
use `as_deref` to avoid allocating a String
ada4a Sep 15, 2025
2d19376
`RustcFormat::store_version`: take params by reference
ada4a Sep 14, 2025
4ea6b20
`config`: introduce `MaybeConfig` type alias
ada4a Sep 14, 2025
42e4037
avoid eager/useless `format!`s
ada4a Sep 14, 2025
8018914
replace `std::iter::once(_)` with `[_]`
ada4a Sep 14, 2025
93ad1fa
use `Itertools::format`
ada4a Sep 14, 2025
5930e34
replace `.ok_or_else(|| anyhow::anyhow!` with `.context(`
ada4a Sep 14, 2025
3a1133f
don't destructure tuple for comparison
ada4a Sep 14, 2025
929f3d4
`expect` some lints
ada4a Sep 16, 2025
cee3a6a
clippy::field_reassign_with_default
ada4a Sep 16, 2025
284ba8c
clippy::vec_init_then_push
ada4a Sep 16, 2025
ac4d7df
avoid `clippy::partialeq_to_none`
ada4a Sep 16, 2025
b722c5f
clippy::manual_find
ada4a Sep 16, 2025
4382423
clippy::ptr_arg
ada4a Sep 16, 2025
7443c6c
deref `Label` to `&str` directly
ada4a Sep 16, 2025
8031a82
clippy::redundant_pattern_matching
ada4a Sep 16, 2025
9b541d0
lints on FCP-related items
ada4a Sep 16, 2025
45d88f8
clippy::unused_self
ada4a Sep 16, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 4 additions & 5 deletions src/gh_range_diff.rs
Original file line number Diff line number Diff line change
Expand Up @@ -446,7 +446,6 @@ struct HtmlDiffPrinter<'a>(pub &'a Interner<&'a str>);

impl HtmlDiffPrinter<'_> {
fn handle_hunk_line<'a>(
&self,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we just allow clippy::unused_self here, we may use &self here in the future.

mut f: impl fmt::Write,
hunk_token_status: HunkTokenStatus,
words: impl Iterator<Item = (&'a str, bool)>,
Expand Down Expand Up @@ -554,7 +553,7 @@ impl UnifiedDiffPrinter for HtmlDiffPrinter<'_> {

// Process all before lines first
for (diff, input) in &diffs_and_inputs {
self.handle_hunk_line(
Self::handle_hunk_line(
&mut f,
HunkTokenStatus::Removed,
input.before.iter().enumerate().map(|(b_pos, b_token)| {
Expand All @@ -565,7 +564,7 @@ impl UnifiedDiffPrinter for HtmlDiffPrinter<'_> {

// Then process all after lines
for (diff, input) in &diffs_and_inputs {
self.handle_hunk_line(
Self::handle_hunk_line(
&mut f,
HunkTokenStatus::Added,
input.after.iter().enumerate().map(|(a_pos, a_token)| {
Expand All @@ -579,7 +578,7 @@ impl UnifiedDiffPrinter for HtmlDiffPrinter<'_> {
if let Some(&last) = before.last() {
for &token in before {
let token = self.0[token];
self.handle_hunk_line(
Self::handle_hunk_line(
&mut f,
HunkTokenStatus::Removed,
std::iter::once((token, false)),
Expand All @@ -593,7 +592,7 @@ impl UnifiedDiffPrinter for HtmlDiffPrinter<'_> {
if let Some(&last) = after.last() {
for &token in after {
let token = self.0[token];
self.handle_hunk_line(
Self::handle_hunk_line(
&mut f,
HunkTokenStatus::Added,
std::iter::once((token, false)),
Expand Down
Loading