Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Commit 199a630

Browse files
committed
order collective benchmark params alphabetically to get a consistent ordering
1 parent a6822b0 commit 199a630

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

frame/collective/src/benchmarking.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,8 @@ benchmarks_instance! {
111111
}
112112

113113
execute {
114-
let m in 1 .. MAX_MEMBERS;
115114
let b in 1 .. MAX_BYTES;
115+
let m in 1 .. MAX_MEMBERS;
116116

117117
let bytes_in_storage = b + size_of::<u32>() as u32;
118118

@@ -141,8 +141,8 @@ benchmarks_instance! {
141141

142142
// This tests when execution would happen immediately after proposal
143143
propose_execute {
144-
let m in 1 .. MAX_MEMBERS;
145144
let b in 1 .. MAX_BYTES;
145+
let m in 1 .. MAX_MEMBERS;
146146

147147
let bytes_in_storage = b + size_of::<u32>() as u32;
148148

@@ -172,9 +172,9 @@ benchmarks_instance! {
172172

173173
// This tests when proposal is created and queued as "proposed"
174174
propose_proposed {
175+
let b in 1 .. MAX_BYTES;
175176
let m in 2 .. MAX_MEMBERS;
176177
let p in 1 .. T::MaxProposals::get();
177-
let b in 1 .. MAX_BYTES;
178178

179179
let bytes_in_storage = b + size_of::<u32>() as u32;
180180

@@ -287,10 +287,10 @@ benchmarks_instance! {
287287
}
288288

289289
close_early_disapproved {
290+
let b in 1 .. MAX_BYTES;
290291
// We choose 4 as a minimum so we always trigger a vote in the voting loop (`for j in ...`)
291292
let m in 4 .. MAX_MEMBERS;
292293
let p in 1 .. T::MaxProposals::get();
293-
let b in 1 .. MAX_BYTES;
294294

295295
let bytes_in_storage = b + size_of::<u32>() as u32;
296296

@@ -364,10 +364,10 @@ benchmarks_instance! {
364364
}
365365

366366
close_early_approved {
367+
let b in 1 .. MAX_BYTES;
367368
// We choose 4 as a minimum so we always trigger a vote in the voting loop (`for j in ...`)
368369
let m in 4 .. MAX_MEMBERS;
369370
let p in 1 .. T::MaxProposals::get();
370-
let b in 1 .. MAX_BYTES;
371371

372372
let bytes_in_storage = b + size_of::<u32>() as u32;
373373

@@ -445,10 +445,10 @@ benchmarks_instance! {
445445
}
446446

447447
close_disapproved {
448+
let b in 1 .. MAX_BYTES;
448449
// We choose 4 as a minimum so we always trigger a vote in the voting loop (`for j in ...`)
449450
let m in 4 .. MAX_MEMBERS;
450451
let p in 1 .. T::MaxProposals::get();
451-
let b in 1 .. MAX_BYTES;
452452

453453
let bytes_in_storage = b + size_of::<u32>() as u32;
454454

@@ -517,10 +517,10 @@ benchmarks_instance! {
517517
}
518518

519519
close_approved {
520+
let b in 1 .. MAX_BYTES;
520521
// We choose 4 as a minimum so we always trigger a vote in the voting loop (`for j in ...`)
521522
let m in 4 .. MAX_MEMBERS;
522523
let p in 1 .. T::MaxProposals::get();
523-
let b in 1 .. MAX_BYTES;
524524

525525
let bytes_in_storage = b + size_of::<u32>() as u32;
526526

0 commit comments

Comments
 (0)