Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Uses next_back() #32478

Merged
merged 1 commit into from
Jul 13, 2023
Merged

Conversation

brooksprumo
Copy link
Contributor

Problem

When upgrading Rust to 1.71.0, clippy has new lints around calling .rev().next()1. This can be replaced with .next_back().

Summary of Changes

Use .next_back().

Footnotes

  1. https://rust-lang.github.io/rust-clippy/master/index.html#/manual_next_back

@brooksprumo brooksprumo self-assigned this Jul 13, 2023
@brooksprumo brooksprumo marked this pull request as ready for review July 13, 2023 17:03
Copy link
Contributor

@bw-solana bw-solana left a comment

Choose a reason for hiding this comment

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

LGTM!

@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #32478 (bd33a39) into master (40249cd) will increase coverage by 0.0%.
The diff coverage is 75.0%.

@@           Coverage Diff           @@
##           master   #32478   +/-   ##
=======================================
  Coverage    82.1%    82.1%           
=======================================
  Files         778      778           
  Lines      210200   210200           
=======================================
+ Hits       172591   172671   +80     
+ Misses      37609    37529   -80     

@brooksprumo brooksprumo merged commit 3c825f2 into solana-labs:master Jul 13, 2023
@brooksprumo brooksprumo deleted the clippy/next-back branch July 13, 2023 17:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants