{"action":"created","issue":{"url":"https://api.github.com/repos/r-spatial/sf/issues/822","repository_url":"https://api.github.com/repos/r-spatial/sf","labels_url":"https://api.github.com/repos/r-spatial/sf/issues/822/labels{/name}","comments_url":"https://api.github.com/repos/r-spatial/sf/issues/822/comments","events_url":"https://api.github.com/repos/r-spatial/sf/issues/822/events","html_url":"https://github.com/r-spatial/sf/issues/822","id":350531015,"node_id":"MDU6SXNzdWUzNTA1MzEwMTU=","number":822,"title":"Memory leak on GEOS exception or user interrupt","user":{"login":"dbaston","id":6318931,"node_id":"MDQ6VXNlcjYzMTg5MzE=","avatar_url":"https://avatars0.githubusercontent.com/u/6318931?v=4","gravatar_id":"","url":"https://api.github.com/users/dbaston","html_url":"https://github.com/dbaston","followers_url":"https://api.github.com/users/dbaston/followers","following_url":"https://api.github.com/users/dbaston/following{/other_user}","gists_url":"https://api.github.com/users/dbaston/gists{/gist_id}","starred_url":"https://api.github.com/users/dbaston/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dbaston/subscriptions","organizations_url":"https://api.github.com/users/dbaston/orgs","repos_url":"https://api.github.com/users/dbaston/repos","events_url":"https://api.github.com/users/dbaston/events{/privacy}","received_events_url":"https://api.github.com/users/dbaston/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":25,"created_at":"2018-08-14T17:49:47Z","updated_at":"2018-08-21T20:56:16Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"The `geometries_from_sfc` function returns an array of raw `GEOSGeometry*` pointers. If a GEOS operation throws an exception or is interrupted by the user, the memory pointed to will not be released. Having `geometries_from_sfc` return an vector of `unique_ptr<GEOSGeometry>` would ensure that memory is not leaked in these cases.\r\n\r\nI can probably make a PR for this, but wanted to check for support first."},"comment":{"url":"https://api.github.com/repos/r-spatial/sf/issues/comments/414818968","html_url":"https://github.com/r-spatial/sf/issues/822#issuecomment-414818968","issue_url":"https://api.github.com/repos/r-spatial/sf/issues/822","id":414818968,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNDgxODk2OA==","user":{"login":"rsbivand","id":10198404,"node_id":"MDQ6VXNlcjEwMTk4NDA0","avatar_url":"https://avatars0.githubusercontent.com/u/10198404?v=4","gravatar_id":"","url":"https://api.github.com/users/rsbivand","html_url":"https://github.com/rsbivand","followers_url":"https://api.github.com/users/rsbivand/followers","following_url":"https://api.github.com/users/rsbivand/following{/other_user}","gists_url":"https://api.github.com/users/rsbivand/gists{/gist_id}","starred_url":"https://api.github.com/users/rsbivand/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rsbivand/subscriptions","organizations_url":"https://api.github.com/users/rsbivand/orgs","repos_url":"https://api.github.com/users/rsbivand/repos","events_url":"https://api.github.com/users/rsbivand/events{/privacy}","received_events_url":"https://api.github.com/users/rsbivand/received_events","type":"User","site_admin":false},"created_at":"2018-08-21T20:56:16Z","updated_at":"2018-08-21T20:56:16Z","author_association":"MEMBER","body":"Thanks, I'll look. I agree that a solution in C is not directly relevant, but gives a within-R basis for comparison (on a slightly different default precision model)."}}
{"actor":{"display_login":"rsbivand"}}