voroskoi/exercism - Change HX6XV5DWSZUH6O5IRJGEOFBE3IFK247N7PFBIFG52CBC4YMTZAVQC

My solutions for exercism.org tracks

scale_generator: make it faster: 1) use string.Title only once, not in the range 2) allocate the string, do not append all the time

Created by voroskoi on December 19, 2020

HX6XV5DWSZUH6O5IRJGEOFBE3IFK247N7PFBIFG52CBC4YMTZAVQC

Dependencies

In channels

main

Change contents

Insertion in go/scale-generator/scale_generator.go at line 26 [4.40]
[3.721]
[4.1076]
```
	tonic = strings.Title(tonic)
```
Replacement in go/scale-generator/scale_generator.go at line 28 [4.40]
B:BD[4.1103] → [3.722:755]
```
		if v == strings.Title(tonic) {
```
[4.1103]
[3.755]
```
		if v == tonic {
```

Replacement in go/scale-generator/scale_generator.go at line 34 [4.40]

B:BD[3.861] → [3.861:900]

B:BD[3.900] → [2.698:740]

			result = append(result, scale[act])
			for i := 0; i < len(interval)-1; i++ {

[3.861]

[2.740]

			result = make([]string, len(interval))
			for i, step := range interval {
				result[i] = scale[act%12]

Replacement in go/scale-generator/scale_generator.go at line 38 [4.40]
B:BD[2.788] → [2.788:817]
```
				case interval[i] == 'm':
```
[2.788]
[3.984]
```
				case step == 'm':
```
Replacement in go/scale-generator/scale_generator.go at line 40 [4.40]
B:BD[3.998] → [2.818:847]
```
				case interval[i] == 'M':
```
[3.998]
[3.1027]
```
				case step == 'M':
```
Replacement in go/scale-generator/scale_generator.go at line 42 [4.40]
B:BD[3.1041] → [2.848:877]
```
				case interval[i] == 'A':
```
[3.1041]
[3.1070]
```
				case step == 'A':
```

Deletion in go/scale-generator/scale_generator.go at line 45 [4.40]

B:BD[3.1090] → [3.1090:1191]

				if act >= len(scale) {
					act = act - len(scale)
				}
				result = append(result, scale[act])