go-graphite / carbonapi

Value assigned to a variable is never read before being overwritten SCC-SA4006
Bug risk
Major
4 months ago4 years old
 45	refetch := false
 46	switch e.Arg(1).Type() {
 47	case parser.EtConst:
 48		windowPoints, err = e.GetIntArg(1) 49		argstr = strconv.Itoa(windowPoints)
 50		if windowPoints < 0 {
 51			// we only care about the absolute value
this value of denominator is never used
386	if changed || len(numerator.Values) != len(denominator.Values) {
387		alignedSeries := ScaleToCommonStep(types.CopyMetricDataSlice(pair), step)
388		numerator = alignedSeries[0]
389		denominator = alignedSeries[1]390
391		return alignedSeries[0], alignedSeries[1]
392	}
this value of numerator is never used
385	step, changed := GetCommonStep(pair)
386	if changed || len(numerator.Values) != len(denominator.Values) {
387		alignedSeries := ScaleToCommonStep(types.CopyMetricDataSlice(pair), step)
388		numerator = alignedSeries[0]389		denominator = alignedSeries[1]
390
391		return alignedSeries[0], alignedSeries[1]