Skip to content

Commit 888c078

Browse files
committed
chore: use positional arg with default value
1 parent c4c5a4b commit 888c078

File tree

2 files changed

+7
-12
lines changed

2 files changed

+7
-12
lines changed

lib/syskit/network_generation/dataflow_dynamics.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -575,7 +575,7 @@ def compute_policies_from(connection_graph, source_task, policy_graph = {})
575575
fallback_policy = explicit_policy.delete(:fallback_policy)
576576
h[port_pair] = policy_for(
577577
source_task, *port_pair, sink_task,
578-
fallback_policy, explicit_policy: explicit_policy
578+
fallback_policy, explicit_policy
579579
)
580580
end
581581
policy_graph[[source_task, sink_task]] = computed_policies
@@ -632,7 +632,7 @@ def policy_compute_data_element(
632632

633633
def policy_for(
634634
source_task, source_port_name, sink_port_name, sink_task,
635-
fallback_policy, explicit_policy: {}
635+
fallback_policy, explicit_policy={}
636636
)
637637
computed_policy =
638638
if explicit_policy[:type]

test/network_generation/test_dataflow_dynamics.rb

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -232,8 +232,7 @@ module NetworkGeneration
232232
task0.out_port.connect_to(task1.in_port)
233233

234234
@dynamics.should_receive(:policy_for)
235-
.with(task0, "out", "in", task1, nil,
236-
explicit_policy: {})
235+
.with(task0, "out", "in", task1, nil, {})
237236
.and_return(type: :buffer, size: 42)
238237
policy_graph = @dynamics.compute_connection_policies
239238

@@ -253,8 +252,7 @@ module NetworkGeneration
253252
task0.out_port.connect_to(task1.in_port)
254253

255254
@dynamics.should_receive(:policy_for)
256-
.with(task0, "out", "in", task1, nil,
257-
explicit_policy: {})
255+
.with(task0, "out", "in", task1, nil, {})
258256
.and_return(type: :buffer, size: 42, init: true)
259257
policy_graph = @dynamics.compute_connection_policies
260258

@@ -274,8 +272,7 @@ module NetworkGeneration
274272
task0.out_port.connect_to(task1.in_port)
275273

276274
@dynamics.should_receive(:policy_for)
277-
.with(task0, "out", "in", task1, nil,
278-
explicit_policy: {})
275+
.with(task0, "out", "in", task1, nil, {})
279276
.and_return(type: :buffer, size: 42, init: false)
280277
policy_graph = @dynamics.compute_connection_policies
281278

@@ -293,8 +290,7 @@ module NetworkGeneration
293290
cmp.c_child.out_port.connect_to(task.in_port)
294291

295292
@dynamics.should_receive(:policy_for)
296-
.with(cmp.c_child, "out", "in", task, nil,
297-
explicit_policy: {})
293+
.with(cmp.c_child, "out", "in", task, nil, {})
298294
.and_return(type: :buffer, size: 42)
299295
policy_graph = @dynamics.compute_connection_policies
300296

@@ -333,8 +329,7 @@ module NetworkGeneration
333329
)
334330

335331
@dynamics.should_receive(:policy_for)
336-
.with(task0, "out", "in", task1, { type: :data },
337-
explicit_policy: {})
332+
.with(task0, "out", "in", task1, { type: :data }, {})
338333
.and_return(type: :buffer, size: 42, init: nil)
339334

340335
policy_graph = @dynamics.compute_connection_policies

0 commit comments

Comments
 (0)