Merge pull request #137 from m-pavel/errorhandling

paulmach · web-flow · commit 3ecd611dcb56 · 2024-01-08T11:51:31.000-08:00
Do not swallow error cause
diff --git a/encoding/mvt/marshal.go b/encoding/mvt/marshal.go
@@ -87,12 +87,12 @@ func addSingleGeometryFeature(layer *vectortile.Tile_Layer, kve *keyValueEncoder
 		return err
 	}
 	if err != nil {
-		return fmt.Errorf("error encoding geometry: %v", g)
+		return fmt.Errorf("error encoding geometry: %v : %s", g, err.Error())
 	}
 
 	tags, err := encodeProperties(kve, p)
 	if err != nil {
-		return fmt.Errorf("error encoding geometry: %v", g)
+		return fmt.Errorf("error encoding geometry: %v : %s", g, err.Error())
 	}
 
 	layer.Features = append(layer.Features, &vectortile.Tile_Feature{

Original file line number	Diff line number	Diff line change
`@@ -87,12 +87,12 @@ func addSingleGeometryFeature(layer vectortile.Tile_Layer, kve keyValueEncoder`
`87`	`87`	`return err`
`88`	`88`	`}`
`89`	`89`	`if err != nil {`
`90`		`- return fmt.Errorf("error encoding geometry: %v", g)`
	`90`	`+ return fmt.Errorf("error encoding geometry: %v : %s", g, err.Error())`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`tags, err := encodeProperties(kve, p)`
`94`	`94`	`if err != nil {`
`95`		`- return fmt.Errorf("error encoding geometry: %v", g)`
	`95`	`+ return fmt.Errorf("error encoding geometry: %v : %s", g, err.Error())`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`layer.Features = append(layer.Features, &vectortile.Tile_Feature{`