{"comment":{"url":"https://api.github.com/repos/ginggs/openlibm/comments/14003490","html_url":"https://github.com/ginggs/openlibm/commit/49059f7ee6b9d72cdeb962bc34c14d56432ade07#commitcomment-14003490","id":14003490,"user":{"login":"ginggs","id":1970404,"avatar_url":"https://avatars.githubusercontent.com/u/1970404?v=3","gravatar_id":"","url":"https://api.github.com/users/ginggs","html_url":"https://github.com/ginggs","followers_url":"https://api.github.com/users/ginggs/followers","following_url":"https://api.github.com/users/ginggs/following{/other_user}","gists_url":"https://api.github.com/users/ginggs/gists{/gist_id}","starred_url":"https://api.github.com/users/ginggs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ginggs/subscriptions","organizations_url":"https://api.github.com/users/ginggs/orgs","repos_url":"https://api.github.com/users/ginggs/repos","events_url":"https://api.github.com/users/ginggs/events{/privacy}","received_events_url":"https://api.github.com/users/ginggs/received_events","type":"User","site_admin":false},"position":4,"line":112,"path":"Make.inc","commit_id":"49059f7ee6b9d72cdeb962bc34c14d56432ade07","created_at":"2015-10-27T07:49:36Z","updated_at":"2015-10-27T07:49:36Z","body":"There is a comment ```#keep these if statements separate```, but it would make sense to have a sane default, otherwise the clean target in Makefile:\r\n```rm -fr $$dir/*.o $$dir/*.a $$dir/*.$(SHLIB_EXT)*; \\```\r\nevaluates to:\r\n```rm -fr $$dir/*.o $$dir/*.a $$dir/*.*; \\```\r\nand cleans everything!\r\n\r\nI noticed this was changed in openspecfun and it now has a default."}}