Fix overlooked nil (#523)

bill-rich · web-flow · commit 6ee5c579684c · 2022-05-04T15:45:12.000-07:00
* Fix overlooked nil

* Use default case
diff --git a/pkg/engine/engine.go b/pkg/engine/engine.go
@@ -255,6 +255,8 @@ func fragmentFirstLine(chunk *sources.Chunk) (int64, *int64) {
 		fragmentStart = &metadata.Bitbucket.Line
 	case *source_metadatapb.MetaData_Gerrit:
 		fragmentStart = &metadata.Gerrit.Line
+	default:
+		return 0, nil
 	}
 	return *fragmentStart, fragmentStart
 }

Original file line number	Diff line number	Diff line change
`@@ -255,6 +255,8 @@ func fragmentFirstLine(chunk sources.Chunk) (int64, int64) {`
`255`	`255`	`fragmentStart = &metadata.Bitbucket.Line`
`256`	`256`	`case *source_metadatapb.MetaData_Gerrit:`
`257`	`257`	`fragmentStart = &metadata.Gerrit.Line`
	`258`	`+ default:`
	`259`	`+ return 0, nil`
`258`	`260`	`}`
`259`	`261`	`return *fragmentStart, fragmentStart`
`260`	`262`	`}`