{"action":"created","issue":{"url":"https://api.github.com/repos/diedricm/ImagePositionLocator/issues/6","labels_url":"https://api.github.com/repos/diedricm/ImagePositionLocator/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/diedricm/ImagePositionLocator/issues/6/comments","events_url":"https://api.github.com/repos/diedricm/ImagePositionLocator/issues/6/events","html_url":"https://github.com/diedricm/ImagePositionLocator/pull/6","id":94514104,"number":6,"title":"Use text-based track storage format.","user":{"login":"rdoeffinger","id":5464937,"avatar_url":"https://avatars.githubusercontent.com/u/5464937?v=3","gravatar_id":"","url":"https://api.github.com/users/rdoeffinger","html_url":"https://github.com/rdoeffinger","followers_url":"https://api.github.com/users/rdoeffinger/followers","following_url":"https://api.github.com/users/rdoeffinger/following{/other_user}","gists_url":"https://api.github.com/users/rdoeffinger/gists{/gist_id}","starred_url":"https://api.github.com/users/rdoeffinger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rdoeffinger/subscriptions","organizations_url":"https://api.github.com/users/rdoeffinger/orgs","repos_url":"https://api.github.com/users/rdoeffinger/repos","events_url":"https://api.github.com/users/rdoeffinger/events{/privacy}","received_events_url":"https://api.github.com/users/rdoeffinger/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-11T22:54:06Z","updated_at":"2015-07-12T18:47:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/diedricm/ImagePositionLocator/pulls/6","html_url":"https://github.com/diedricm/ImagePositionLocator/pull/6","diff_url":"https://github.com/diedricm/ImagePositionLocator/pull/6.diff","patch_url":"https://github.com/diedricm/ImagePositionLocator/pull/6.patch"},"body":"Maybe not very Java-style, but much easier to process\r\nin other applications, to debug and more robust -\r\ne.g. no issues when renaming the classes.\r\nAlso a lot smaller for typical marker file sizes.\r\nPotentially more maintenance effort though if the\r\nstructure changes significantly in the future.\r\n\r\nSigned-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>"},"comment":{"url":"https://api.github.com/repos/diedricm/ImagePositionLocator/issues/comments/120753813","html_url":"https://github.com/diedricm/ImagePositionLocator/pull/6#issuecomment-120753813","issue_url":"https://api.github.com/repos/diedricm/ImagePositionLocator/issues/6","id":120753813,"user":{"login":"rdoeffinger","id":5464937,"avatar_url":"https://avatars.githubusercontent.com/u/5464937?v=3","gravatar_id":"","url":"https://api.github.com/users/rdoeffinger","html_url":"https://github.com/rdoeffinger","followers_url":"https://api.github.com/users/rdoeffinger/followers","following_url":"https://api.github.com/users/rdoeffinger/following{/other_user}","gists_url":"https://api.github.com/users/rdoeffinger/gists{/gist_id}","starred_url":"https://api.github.com/users/rdoeffinger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rdoeffinger/subscriptions","organizations_url":"https://api.github.com/users/rdoeffinger/orgs","repos_url":"https://api.github.com/users/rdoeffinger/repos","events_url":"https://api.github.com/users/rdoeffinger/events{/privacy}","received_events_url":"https://api.github.com/users/rdoeffinger/received_events","type":"User","site_admin":false},"created_at":"2015-07-12T18:47:53Z","updated_at":"2015-07-12T18:47:53Z","body":"I have to admit, my thinking is rather the other way round: I'd only use a binary format if using text is too much of a pain.\r\nIn this case the data contained is simple enough that it being viewable and editable in my opinions a plus. It also has the advantage that supporting leaving out the \"time\" fields for example (which seem fairly pointless) is easy to do and detect, binary formats are usually more likely to need to bump the revision number and thus need extra code for forward/backward compatibility.\r\nAnd as said, the debug part. If I had been able to actually read the track files it might have saved me a few hours debugging (admittedly because debugging on Android at least for a newcomer is a huge pain).\r\nNot that I particularly mind a binary format, but at least not having any Java-specifics and being able to understand it with the help of a hex editor would be nice...\r\nOtherwise I'd be back to having to think about adding an \"export\" feature for something that can interoperate with non-Java-based programs (no concrete plans yet, but I might want a desktop based viewer/editor at some point. And I really, really, don't want anything Java based for that)."}}