Skip to content

Commit 54c8a61

Browse files
aaronbuchwaldatif-konasl
authored andcommitted
core/vm: replace repeated string with variable in tests (ethereum#22774)
1 parent 12794bb commit 54c8a61

File tree

1 file changed

+35
-34
lines changed

1 file changed

+35
-34
lines changed

core/vm/instructions_test.go

Lines changed: 35 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ type twoOperandParams struct {
4040
y string
4141
}
4242

43+
var alphabetSoup = "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
4344
var commonParams []*twoOperandParams
4445
var twoOpMethods map[string]executionFunc
4546

@@ -347,8 +348,8 @@ func BenchmarkOpSub256(b *testing.B) {
347348
}
348349

349350
func BenchmarkOpMul(b *testing.B) {
350-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
351-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
351+
x := alphabetSoup
352+
y := alphabetSoup
352353

353354
opBenchmark(b, opMul, x, y)
354355
}
@@ -379,64 +380,64 @@ func BenchmarkOpSdiv(b *testing.B) {
379380
}
380381

381382
func BenchmarkOpMod(b *testing.B) {
382-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
383-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
383+
x := alphabetSoup
384+
y := alphabetSoup
384385

385386
opBenchmark(b, opMod, x, y)
386387
}
387388

388389
func BenchmarkOpSmod(b *testing.B) {
389-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
390-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
390+
x := alphabetSoup
391+
y := alphabetSoup
391392

392393
opBenchmark(b, opSmod, x, y)
393394
}
394395

395396
func BenchmarkOpExp(b *testing.B) {
396-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
397-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
397+
x := alphabetSoup
398+
y := alphabetSoup
398399

399400
opBenchmark(b, opExp, x, y)
400401
}
401402

402403
func BenchmarkOpSignExtend(b *testing.B) {
403-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
404-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
404+
x := alphabetSoup
405+
y := alphabetSoup
405406

406407
opBenchmark(b, opSignExtend, x, y)
407408
}
408409

409410
func BenchmarkOpLt(b *testing.B) {
410-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
411-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
411+
x := alphabetSoup
412+
y := alphabetSoup
412413

413414
opBenchmark(b, opLt, x, y)
414415
}
415416

416417
func BenchmarkOpGt(b *testing.B) {
417-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
418-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
418+
x := alphabetSoup
419+
y := alphabetSoup
419420

420421
opBenchmark(b, opGt, x, y)
421422
}
422423

423424
func BenchmarkOpSlt(b *testing.B) {
424-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
425-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
425+
x := alphabetSoup
426+
y := alphabetSoup
426427

427428
opBenchmark(b, opSlt, x, y)
428429
}
429430

430431
func BenchmarkOpSgt(b *testing.B) {
431-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
432-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
432+
x := alphabetSoup
433+
y := alphabetSoup
433434

434435
opBenchmark(b, opSgt, x, y)
435436
}
436437

437438
func BenchmarkOpEq(b *testing.B) {
438-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
439-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
439+
x := alphabetSoup
440+
y := alphabetSoup
440441

441442
opBenchmark(b, opEq, x, y)
442443
}
@@ -446,45 +447,45 @@ func BenchmarkOpEq2(b *testing.B) {
446447
opBenchmark(b, opEq, x, y)
447448
}
448449
func BenchmarkOpAnd(b *testing.B) {
449-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
450-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
450+
x := alphabetSoup
451+
y := alphabetSoup
451452

452453
opBenchmark(b, opAnd, x, y)
453454
}
454455

455456
func BenchmarkOpOr(b *testing.B) {
456-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
457-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
457+
x := alphabetSoup
458+
y := alphabetSoup
458459

459460
opBenchmark(b, opOr, x, y)
460461
}
461462

462463
func BenchmarkOpXor(b *testing.B) {
463-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
464-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
464+
x := alphabetSoup
465+
y := alphabetSoup
465466

466467
opBenchmark(b, opXor, x, y)
467468
}
468469

469470
func BenchmarkOpByte(b *testing.B) {
470-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
471-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
471+
x := alphabetSoup
472+
y := alphabetSoup
472473

473474
opBenchmark(b, opByte, x, y)
474475
}
475476

476477
func BenchmarkOpAddmod(b *testing.B) {
477-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
478-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
479-
z := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
478+
x := alphabetSoup
479+
y := alphabetSoup
480+
z := alphabetSoup
480481

481482
opBenchmark(b, opAddmod, x, y, z)
482483
}
483484

484485
func BenchmarkOpMulmod(b *testing.B) {
485-
x := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
486-
y := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
487-
z := "ABCDEF090807060504030201ffffffffffffffffffffffffffffffffffffffff"
486+
x := alphabetSoup
487+
y := alphabetSoup
488+
z := alphabetSoup
488489

489490
opBenchmark(b, opMulmod, x, y, z)
490491
}

0 commit comments

Comments
 (0)