Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit c9f42e3

Browse files
eileencodeskddnewton
authored andcommitted
remove kwargs on Breakable
1 parent 09afd69 commit c9f42e3

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

lib/syntax_tree/prism.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3264,7 +3264,7 @@ def visit_elements_spaced(elements, comments)
32643264
def visit_heredoc(node, parts)
32653265
# This is a very specific behavior where you want to force a newline, but
32663266
# don't want to force the break parent.
3267-
separator = PrettierPrint::Breakable.new(" ", 1, indent: false, force: true)
3267+
separator = PrettierPrint::Breakable.new(" ", 1, true, false)
32683268
opening = node.opening
32693269

32703270
# If the heredoc is indented, then we're going to need to reintroduce the

lib/syntax_tree/prism/prettier_print.rb

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,8 @@ class Breakable
8383
def initialize(
8484
separator = " ",
8585
width = separator.length,
86-
force: false,
87-
indent: true
86+
force = false,
87+
indent = true
8888
)
8989
@separator = separator
9090
@width = width
@@ -107,10 +107,10 @@ def type
107107

108108
# Below here are the most common combination of options that are created when
109109
# creating new breakables. They are here to cut down on some allocations.
110-
BREAKABLE_SPACE = Breakable.new(" ", 1, indent: true, force: false).freeze
111-
BREAKABLE_EMPTY = Breakable.new("", 0, indent: true, force: false).freeze
112-
BREAKABLE_FORCE = Breakable.new(" ", 1, indent: true, force: true).freeze
113-
BREAKABLE_RETURN = Breakable.new(" ", 1, indent: false, force: true).freeze
110+
BREAKABLE_SPACE = Breakable.new(" ", 1, false, true).freeze
111+
BREAKABLE_EMPTY = Breakable.new("", 0, false, true).freeze
112+
BREAKABLE_FORCE = Breakable.new(" ", 1, true, true).freeze
113+
BREAKABLE_RETURN = Breakable.new(" ", 1, true, false).freeze
114114

115115
# A node in the print tree that forces the surrounding group to print out in
116116
# the "break" mode as opposed to the "flat" mode. Useful for when you need to
@@ -688,7 +688,7 @@ def breakable(
688688
indent: true,
689689
force: false
690690
)
691-
target << Breakable.new(separator, width, indent: indent, force: !!force)
691+
target << Breakable.new(separator, width, !!force, indent)
692692
break_parent if force == true
693693
end
694694

0 commit comments

Comments
 (0)