{"issue":{"pull_request":{"diff_url":null,"html_url":null,"patch_url":null},"state":"open","user":{"avatar_url":"https://secure.gravatar.com/avatar/285043bb1b796a1687d10e1549aaef99?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"285043bb1b796a1687d10e1549aaef99","url":"https://api.github.com/users/jaydenmorris","login":"jaydenmorris","id":1178033},"milestone":null,"assignee":null,"updated_at":"2011-11-07T17:34:34Z","labels":[],"comments":1,"closed_at":null,"url":"https://api.github.com/repos/robertpenner/as3-signals/issues/59","body":"I think I have come across a bug in the priority signal. I did a series of unit tests to narrow it down and this one failed to fill 5 slots.\r\n\r\n[Test]\r\npublic function positive_to_negative_priority_values():void\r\n{\r\n\t_signal.addWithPriority(testFunction_1, 1);\r\n\tAssert.assertEquals(_signal.numListeners, 1);\r\n\t\r\n\t_signal.addWithPriority(testFunction_2, 0);\r\n\tAssert.assertEquals(_signal.numListeners, 2);\r\n\t\r\n\t_signal.addWithPriority(testFunction_3, -1);\r\n\tAssert.assertEquals(_signal.numListeners, 3);\r\n\t\r\n\t_signal.addWithPriority(testFunction_4, 0);\r\n\tAssert.assertEquals(_signal.numListeners, 4);\r\n\t\r\n\t_signal.addWithPriority(testFunction_5, 1);\r\n\tAssert.assertEquals(_signal.numListeners, 5);\r\n}\r\n\r\nif you use 1, 0 then -1 as signal priorities in that order in a proper test case it should be reproducible.\r\n\r\nI love signals btw. building a game with them :)","html_url":"https://github.com/robertpenner/as3-signals/issues/59","number":59,"title":"priority signal bug","id":2163050,"created_at":"2011-11-07T14:31:40Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/6bd7b1f57be5d2b9d5aefc57c9b10e89?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"6bd7b1f57be5d2b9d5aefc57c9b10e89","url":"https://api.github.com/users/stickupkid","login":"stickupkid","id":104714},"updated_at":"2011-11-07T17:34:34Z","url":"https://api.github.com/repos/robertpenner/as3-signals/issues/comments/2656568","body":"I can't replicate this, this my diff (sorry it's not clean, I pulled from the upstream at the same time)\r\n\r\nhttps://github.com/stickupkid/as3-signals/commit/b6c19b396aee01885ed9a685414dd4362398df7c","id":2656568,"created_at":"2011-11-07T17:34:34Z"}}