{"action":"created","issue":{"url":"https://api.github.com/repos/MrNex/NGen/issues/17","labels_url":"https://api.github.com/repos/MrNex/NGen/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/MrNex/NGen/issues/17/comments","events_url":"https://api.github.com/repos/MrNex/NGen/issues/17/events","html_url":"https://github.com/MrNex/NGen/issues/17","id":93825744,"number":17,"title":"Code Janitoring - Reduce all lines to 80 chars in length to improve readiblity","user":{"login":"Waqar144","id":7696024,"avatar_url":"https://avatars.githubusercontent.com/u/7696024?v=3","gravatar_id":"","url":"https://api.github.com/users/Waqar144","html_url":"https://github.com/Waqar144","followers_url":"https://api.github.com/users/Waqar144/followers","following_url":"https://api.github.com/users/Waqar144/following{/other_user}","gists_url":"https://api.github.com/users/Waqar144/gists{/gist_id}","starred_url":"https://api.github.com/users/Waqar144/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Waqar144/subscriptions","organizations_url":"https://api.github.com/users/Waqar144/orgs","repos_url":"https://api.github.com/users/Waqar144/repos","events_url":"https://api.github.com/users/Waqar144/events{/privacy}","received_events_url":"https://api.github.com/users/Waqar144/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-08T15:57:42Z","updated_at":"2015-07-08T16:04:25Z","closed_at":null,"body":"Since I use Vi, and edit in a window with width set to 80. The code often looks a bit messy.\r\nIs it a good idea or maybe just leave the code base as it is."},"comment":{"url":"https://api.github.com/repos/MrNex/NGen/issues/comments/119636437","html_url":"https://github.com/MrNex/NGen/issues/17#issuecomment-119636437","issue_url":"https://api.github.com/repos/MrNex/NGen/issues/17","id":119636437,"user":{"login":"MrNex","id":3581020,"avatar_url":"https://avatars.githubusercontent.com/u/3581020?v=3","gravatar_id":"","url":"https://api.github.com/users/MrNex","html_url":"https://github.com/MrNex","followers_url":"https://api.github.com/users/MrNex/followers","following_url":"https://api.github.com/users/MrNex/following{/other_user}","gists_url":"https://api.github.com/users/MrNex/gists{/gist_id}","starred_url":"https://api.github.com/users/MrNex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MrNex/subscriptions","organizations_url":"https://api.github.com/users/MrNex/orgs","repos_url":"https://api.github.com/users/MrNex/repos","events_url":"https://api.github.com/users/MrNex/events{/privacy}","received_events_url":"https://api.github.com/users/MrNex/received_events","type":"User","site_admin":false},"created_at":"2015-07-08T16:04:25Z","updated_at":"2015-07-08T16:04:25Z","body":"I do not believe a fixed width is what we want, however I do agree some lines may be too long.\r\n\r\nWhat we should do, instead of hardlimiting to 80 characters-- we should take longer lines and split them based on readability.\r\n\r\nFor example:\r\nvoid DoSomething(float scalar, Vector v1, Vector v2, Vector v3, Vector v4, Matrix m1, Matrix m2, Matrix m3);\r\nMight become\r\n```\r\nvoid DoSomething( \r\n  float Scalar,\r\n  Vector v1, Vector v2, Vector v3, Vector v4,\r\n  Matrix m1, Matrix m2, Matrix m3\r\n);\r\n```\r\n\r\nRather than\r\n```\r\nvoid DoSomething( float Scalar, Vector v1, Vector v2,\r\n  Vector v3, Vector v4, Matrix m1, Matrix m2, Matrix m3,\r\n  Matrix m4);\r\n```"}}