{"action":"created","issue":{"url":"https://api.github.com/repos/HaxeFlixel/flixel/issues/1770","repository_url":"https://api.github.com/repos/HaxeFlixel/flixel","labels_url":"https://api.github.com/repos/HaxeFlixel/flixel/issues/1770/labels{/name}","comments_url":"https://api.github.com/repos/HaxeFlixel/flixel/issues/1770/comments","events_url":"https://api.github.com/repos/HaxeFlixel/flixel/issues/1770/events","html_url":"https://github.com/HaxeFlixel/flixel/pull/1770","id":139645785,"number":1770,"title":"Fast floor","user":{"login":"MSGhero","id":5033927,"avatar_url":"https://avatars.githubusercontent.com/u/5033927?v=3","gravatar_id":"","url":"https://api.github.com/users/MSGhero","html_url":"https://github.com/MSGhero","followers_url":"https://api.github.com/users/MSGhero/followers","following_url":"https://api.github.com/users/MSGhero/following{/other_user}","gists_url":"https://api.github.com/users/MSGhero/gists{/gist_id}","starred_url":"https://api.github.com/users/MSGhero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MSGhero/subscriptions","organizations_url":"https://api.github.com/users/MSGhero/orgs","repos_url":"https://api.github.com/users/MSGhero/repos","events_url":"https://api.github.com/users/MSGhero/events{/privacy}","received_events_url":"https://api.github.com/users/MSGhero/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2016-03-09T17:23:38Z","updated_at":"2016-03-09T17:53:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/HaxeFlixel/flixel/pulls/1770","html_url":"https://github.com/HaxeFlixel/flixel/pull/1770","diff_url":"https://github.com/HaxeFlixel/flixel/pull/1770.diff","patch_url":"https://github.com/HaxeFlixel/flixel/pull/1770.patch"},"body":"It kinda looks like the function call to `Math.floor` takes longer than actually flooring on js. `fastFloor` was about 20x faster in my little test suite with analyzer off.\r\n\r\nNot sure if there's a `fastCeil`. I was also thinking about using `fastSin` and `fastCos` throughout the library, but that's technically a breaking change?"},"comment":{"url":"https://api.github.com/repos/HaxeFlixel/flixel/issues/comments/194423094","html_url":"https://github.com/HaxeFlixel/flixel/pull/1770#issuecomment-194423094","issue_url":"https://api.github.com/repos/HaxeFlixel/flixel/issues/1770","id":194423094,"user":{"login":"Gama11","id":2620907,"avatar_url":"https://avatars.githubusercontent.com/u/2620907?v=3","gravatar_id":"","url":"https://api.github.com/users/Gama11","html_url":"https://github.com/Gama11","followers_url":"https://api.github.com/users/Gama11/followers","following_url":"https://api.github.com/users/Gama11/following{/other_user}","gists_url":"https://api.github.com/users/Gama11/gists{/gist_id}","starred_url":"https://api.github.com/users/Gama11/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Gama11/subscriptions","organizations_url":"https://api.github.com/users/Gama11/orgs","repos_url":"https://api.github.com/users/Gama11/repos","events_url":"https://api.github.com/users/Gama11/events{/privacy}","received_events_url":"https://api.github.com/users/Gama11/received_events","type":"User","site_admin":false},"created_at":"2016-03-09T17:53:01Z","updated_at":"2016-03-09T17:53:01Z","body":"Let's see what the Haxe unit tests say about this implementation: https://travis-ci.org/Gama11/haxe/builds/114854752\r\n\r\nNot sure if there are a lot of tests for `Math.floor()` though."}}