Skip to content

nitpicks #5886

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
youknowone merged 12 commits into from
Jul 2, 2025
Merged

nitpicks #5886

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
319065e
Use `Self` instead of struct name
ShaharNaveh Jun 30, 2025
8c8ccdc
General fixes
ShaharNaveh Jun 30, 2025
df03be4
General
ShaharNaveh Jun 29, 2025
bcfe02a
General fixes
ShaharNaveh Jul 2, 2025
91b5666
More nitpicks
ShaharNaveh Jul 2, 2025
9aed7aa
Added newlines
ShaharNaveh Jul 2, 2025
d9aeed7
More nitpicks
ShaharNaveh Jul 2, 2025
3992208
Use Self where possible
ShaharNaveh Jul 2, 2025
7a6e815
Use Self where possible
ShaharNaveh Jul 2, 2025
48b332c
Use Self
ShaharNaveh Jul 2, 2025
75ac4c4
Use Self more
ShaharNaveh Jul 2, 2025
da95cca
Use Self
ShaharNaveh Jul 2, 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
Next Next commit
Use Self instead of struct name
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Jul 2, 2025
commit 319065e4f0f0ba91a194b0e19df06ce32d71c235
6 changes: 3 additions & 3 deletions vm/src/readline.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ mod basic_readline {

impl<H: Helper> Readline<H> {
pub fn new(helper: H) -> Self {
Readline { helper }
Self { helper }
}

pub fn load_history(&mut self, _path: &Path) -> OtherResult<()> {
Expand Down Expand Up @@ -86,7 +86,7 @@ mod rustyline_readline {
)
.expect("failed to initialize line editor");
repl.set_helper(Some(helper));
Readline { repl }
Self { repl }
}

pub fn load_history(&mut self, path: &Path) -> OtherResult<()> {
Expand Down Expand Up @@ -136,7 +136,7 @@ pub struct Readline<H: Helper>(readline_inner::Readline<H>);

impl<H: Helper> Readline<H> {
pub fn new(helper: H) -> Self {
Readline(readline_inner::Readline::new(helper))
Self(readline_inner::Readline::new(helper))
}

pub fn load_history(&mut self, path: &Path) -> OtherResult<()> {
Expand Down