{"action":"created","issue":{"url":"https://api.github.com/repos/andreyzhd/VideoMEG/issues/8","labels_url":"https://api.github.com/repos/andreyzhd/VideoMEG/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/andreyzhd/VideoMEG/issues/8/comments","events_url":"https://api.github.com/repos/andreyzhd/VideoMEG/issues/8/events","html_url":"https://github.com/andreyzhd/VideoMEG/pull/8","id":55189324,"number":8,"title":"FIX: Tabs->Spaces and QStrings","user":{"login":"Eric89GXL","id":2365790,"avatar_url":"https://avatars.githubusercontent.com/u/2365790?v=3","gravatar_id":"","url":"https://api.github.com/users/Eric89GXL","html_url":"https://github.com/Eric89GXL","followers_url":"https://api.github.com/users/Eric89GXL/followers","following_url":"https://api.github.com/users/Eric89GXL/following{/other_user}","gists_url":"https://api.github.com/users/Eric89GXL/gists{/gist_id}","starred_url":"https://api.github.com/users/Eric89GXL/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Eric89GXL/subscriptions","organizations_url":"https://api.github.com/users/Eric89GXL/orgs","repos_url":"https://api.github.com/users/Eric89GXL/repos","events_url":"https://api.github.com/users/Eric89GXL/events{/privacy}","received_events_url":"https://api.github.com/users/Eric89GXL/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-22T18:20:35Z","updated_at":"2015-01-22T18:27:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/andreyzhd/VideoMEG/pulls/8","html_url":"https://github.com/andreyzhd/VideoMEG/pull/8","diff_url":"https://github.com/andreyzhd/VideoMEG/pull/8.diff","patch_url":"https://github.com/andreyzhd/VideoMEG/pull/8.patch"},"body":"This does a few things:\r\n\r\n1. Updates the binary. I built it on 14.10 so I need to double-check it works on 14.04.\r\n\r\n2. Changes all tabs to spaces.\r\n\r\n3. Changes out some `char *` instances for `QString` to move toward more simplicity / consistency.\r\n\r\n4. Added a \"Limit display size\" check box that will be useful for us when ensuring things are properly focused. It makes it so that the display size won't exceed the incoming video size when checked.\r\n\r\n5. Makes the videos centered vertically and horizontally (used to be left-justified) in the window.\r\n\r\nI hope I'm not committing too many changes here. Again, feel free to provide feedback about changes if these are not what you want. It's tough to see the substantive changes in the midst of all the tab changes (sorry), so I'll comment in-line where they occurred."},"comment":{"url":"https://api.github.com/repos/andreyzhd/VideoMEG/issues/comments/71071300","html_url":"https://github.com/andreyzhd/VideoMEG/pull/8#issuecomment-71071300","issue_url":"https://api.github.com/repos/andreyzhd/VideoMEG/issues/8","id":71071300,"user":{"login":"Eric89GXL","id":2365790,"avatar_url":"https://avatars.githubusercontent.com/u/2365790?v=3","gravatar_id":"","url":"https://api.github.com/users/Eric89GXL","html_url":"https://github.com/Eric89GXL","followers_url":"https://api.github.com/users/Eric89GXL/followers","following_url":"https://api.github.com/users/Eric89GXL/following{/other_user}","gists_url":"https://api.github.com/users/Eric89GXL/gists{/gist_id}","starred_url":"https://api.github.com/users/Eric89GXL/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Eric89GXL/subscriptions","organizations_url":"https://api.github.com/users/Eric89GXL/orgs","repos_url":"https://api.github.com/users/Eric89GXL/repos","events_url":"https://api.github.com/users/Eric89GXL/events{/privacy}","received_events_url":"https://api.github.com/users/Eric89GXL/received_events","type":"User","site_admin":false},"created_at":"2015-01-22T18:27:19Z","updated_at":"2015-01-22T18:27:19Z","body":"FYI I used this to change the spaces:\r\n```\r\nfind . -name '*.cpp' ! -type d -exec bash -c 'expand -t 4 \"$0\" > /tmp/e && mv /tmp/e \"$0\"' {} \\;\r\nfind . -name '*.h' ! -type d -exec bash -c 'expand -t 4 \"$0\" > /tmp/e && mv /tmp/e \"$0\"' {} \\;\r\n```\r\nThis PR also modifies `.gitignore` to add the build files (`.o`, `ui_*.h`, and `moc_*.cpp`)."}}