Skip to content

Commit 883ee42

Browse files
dfawleyzasweq
authored andcommitted
Consistent param names
1 parent a6a8d82 commit 883ee42

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

stats/metrics.go

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,10 @@ type MetricSet struct {
2828
metrics map[string]bool
2929
}
3030

31-
// NewMetricSet returns a MetricSet containing metrics.
32-
func NewMetricSet(metrics ...string) *MetricSet {
31+
// NewMetricSet returns a MetricSet containing metricNames.
32+
func NewMetricSet(metricNames ...string) *MetricSet {
3333
newMetrics := make(map[string]bool)
34-
for _, metric := range metrics {
34+
for _, metric := range metricNames {
3535
newMetrics[metric] = true
3636
}
3737
return &MetricSet{metrics: newMetrics}
@@ -43,15 +43,15 @@ func (m *MetricSet) Metrics() map[string]bool {
4343
return m.metrics
4444
}
4545

46-
// Add adds the metrics to the metrics set and returns a new copy with the
46+
// Add adds the metricNames to the metrics set and returns a new copy with the
4747
// additional metrics.
48-
func (m *MetricSet) Add(metrics ...string) *MetricSet {
48+
func (m *MetricSet) Add(metricNames ...string) *MetricSet {
4949
newMetrics := make(map[string]bool)
5050
for metric := range m.metrics {
5151
newMetrics[metric] = true
5252
}
5353

54-
for _, metric := range metrics {
54+
for _, metric := range metricNames {
5555
newMetrics[metric] = true
5656
}
5757
return &MetricSet{metrics: newMetrics}
@@ -66,15 +66,15 @@ func (m *MetricSet) Join(metrics *MetricSet) *MetricSet {
6666
return &MetricSet{metrics: newMetrics}
6767
}
6868

69-
// Remove removes the metrics from the metrics set and returns a new copy with
70-
// the metrics removed.
71-
func (m *MetricSet) Remove(metrics ...string) *MetricSet {
69+
// Remove removes the metricNames from the metrics set and returns a new copy
70+
// with the metrics removed.
71+
func (m *MetricSet) Remove(metricNames ...string) *MetricSet {
7272
newMetrics := make(map[string]bool)
7373
for metric := range m.metrics {
7474
newMetrics[metric] = true
7575
}
7676

77-
for _, metric := range metrics {
77+
for _, metric := range metricNames {
7878
delete(newMetrics, metric)
7979
}
8080
return &MetricSet{metrics: newMetrics}

0 commit comments

Comments
 (0)