Skip to content

Commit 5441514

Browse files
authored
Remove unused heredoc checks in render queue (#827)
1 parent 08a000a commit 5441514

File tree

2 files changed

+0
-39
lines changed

2 files changed

+0
-39
lines changed

librubyfmt/src/intermediary.rs

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -34,23 +34,11 @@ impl<'src> Intermediary<'src> {
3434
self.tokens.len()
3535
}
3636

37-
pub fn pop_heredoc_mistake(&mut self) {
38-
self.tokens.remove(self.tokens.len() - 1);
39-
self.tokens.remove(self.tokens.len() - 1);
40-
self.index_of_last_hard_newline = self.tokens.len() - 1;
41-
}
42-
4337
pub fn fix_heredoc_duplicate_indent_mistake(&mut self) {
4438
// Remove duplicate indent
4539
self.tokens.remove(self.tokens.len() - 3);
4640
}
4741

48-
pub fn fix_heredoc_arg_newline_mistake(&mut self) {
49-
// Remove duplicate newline
50-
self.tokens.remove(self.tokens.len() - 1);
51-
self.index_of_last_hard_newline = self.tokens.len() - 1;
52-
}
53-
5442
pub fn last<const N: usize>(&self) -> Option<&[ConcreteLineToken<'src>; N]> {
5543
self.tokens.last_chunk::<N>()
5644
}

librubyfmt/src/render_queue_writer.rs

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -133,18 +133,6 @@ impl<'src> RenderQueueWriter<'src> {
133133
},
134134
}
135135

136-
if let Some(
137-
[
138-
ConcreteLineToken::HeredocClose { .. },
139-
ConcreteLineToken::HardNewLine,
140-
ConcreteLineToken::Indent { .. },
141-
ConcreteLineToken::HardNewLine,
142-
],
143-
) = accum.last::<4>()
144-
{
145-
accum.pop_heredoc_mistake();
146-
}
147-
148136
if let Some(
149137
[
150138
ConcreteLineToken::End,
@@ -210,21 +198,6 @@ impl<'src> RenderQueueWriter<'src> {
210198
{
211199
accum.fix_heredoc_duplicate_indent_mistake();
212200
}
213-
214-
if let Some(
215-
[
216-
ConcreteLineToken::HeredocClose { .. },
217-
ConcreteLineToken::HardNewLine,
218-
ConcreteLineToken::Indent { .. },
219-
ConcreteLineToken::Delim { .. },
220-
ConcreteLineToken::Comma,
221-
ConcreteLineToken::HardNewLine,
222-
ConcreteLineToken::HardNewLine,
223-
],
224-
) = accum.last::<7>()
225-
{
226-
accum.fix_heredoc_arg_newline_mistake();
227-
}
228201
}
229202
}
230203

0 commit comments

Comments
 (0)