crs is lost in st_as_sfc.bbox

[faridcher]

st_as_sfc(st_bbox(st_linestring(matrix(1:4,2))), crs=4326)

[edzer]

Yes, but what if argument crs is different from the CRS of x? Why not use

st_as_sfc(st_set_crs(st_bbox(st_linestring(matrix(1:4,2))), 4326))

[faridcher]

i think st_set_crs is too verbose and redundant. Why not just use st_as_sfc?
As a side note, I think st_bbox(obj, ..., crs) should support objects other than Extent and numeric so st_bbox(st_linestring(matrix(1:4,2)), crs=4326) would work then.

[edzer]

You did not answer my first question.

[faridcher]

You are right. I wrote this patch when st_bbox did not support crs. st_set_crs works here but my above side note
still applies and makes st_bbox more flexible. thank you