{"action":"created","issue":{"url":"https://api.github.com/repos/fredreichbier/genie/issues/13","labels_url":"https://api.github.com/repos/fredreichbier/genie/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/fredreichbier/genie/issues/13/comments","events_url":"https://api.github.com/repos/fredreichbier/genie/issues/13/events","html_url":"https://github.com/fredreichbier/genie/issues/13","id":100432630,"number":13,"title":"Combine `save` and `export` commands.","user":{"login":"Phrohdoh","id":4861023,"avatar_url":"https://avatars.githubusercontent.com/u/4861023?v=3","gravatar_id":"","url":"https://api.github.com/users/Phrohdoh","html_url":"https://github.com/Phrohdoh","followers_url":"https://api.github.com/users/Phrohdoh/followers","following_url":"https://api.github.com/users/Phrohdoh/following{/other_user}","gists_url":"https://api.github.com/users/Phrohdoh/gists{/gist_id}","starred_url":"https://api.github.com/users/Phrohdoh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Phrohdoh/subscriptions","organizations_url":"https://api.github.com/users/Phrohdoh/orgs","repos_url":"https://api.github.com/users/Phrohdoh/repos","events_url":"https://api.github.com/users/Phrohdoh/events{/privacy}","received_events_url":"https://api.github.com/users/Phrohdoh/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-11T23:18:16Z","updated_at":"2015-09-18T18:08:10Z","closed_at":"2015-09-18T18:08:10Z","body":"**Issue**: They do the same thing from a user's standpoint.\r\nYou could say `save` is a subcommand of `export` as it only deals with (interpreted) SLP data.\r\n\r\n```\r\nexport   writes a file to disk\r\nsave     writes a frame to disk\r\n```\r\n\r\n**Proposal**: Combine the two as `export` but call the appropriate methods depending on the passed parameters.\r\n\r\n*Proposed examples*:\r\nFilenames would be appended with `N-of-M` if `frames=N` is specified.\r\n\r\n```\r\nexport 271.slp frames=* foo.slp       => write all frames to disk\r\nexport 271.slp frames=0,5[,n] foo.slp => writes only the specified frames to disk\r\nexport 271.slp foo.slp                => write entire file to disk\r\nexport 50500.bin foo.bin              => write entire file to disk\r\nexport 50500.bin frames=* foo.bin     => write entire file to disk, warn that `frames` is invalid\r\n```\r\n\r\nAny opinions / interjections before I spend a bit of time trying to implement this?"},"comment":{"url":"https://api.github.com/repos/fredreichbier/genie/issues/comments/141522203","html_url":"https://github.com/fredreichbier/genie/issues/13#issuecomment-141522203","issue_url":"https://api.github.com/repos/fredreichbier/genie/issues/13","id":141522203,"user":{"login":"Phrohdoh","id":4861023,"avatar_url":"https://avatars.githubusercontent.com/u/4861023?v=3","gravatar_id":"","url":"https://api.github.com/users/Phrohdoh","html_url":"https://github.com/Phrohdoh","followers_url":"https://api.github.com/users/Phrohdoh/followers","following_url":"https://api.github.com/users/Phrohdoh/following{/other_user}","gists_url":"https://api.github.com/users/Phrohdoh/gists{/gist_id}","starred_url":"https://api.github.com/users/Phrohdoh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Phrohdoh/subscriptions","organizations_url":"https://api.github.com/users/Phrohdoh/orgs","repos_url":"https://api.github.com/users/Phrohdoh/repos","events_url":"https://api.github.com/users/Phrohdoh/events{/privacy}","received_events_url":"https://api.github.com/users/Phrohdoh/received_events","type":"User","site_admin":false},"created_at":"2015-09-18T18:08:10Z","updated_at":"2015-09-18T18:08:10Z","body":"I certainly see your point.\r\nThat ought to be a fine change, but then I feel I'd be making a change for the sake of changing something, which I am not a fan of.\r\n\r\nClosing as there is nothing more to do here."}}