-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rullzer prevent inf loop #146
Conversation
10a081f
to
f652b5d
Compare
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
f652b5d
to
771f945
Compare
Did you check the test will fail without the patch? |
Thanks for taking over @DeepDiver1975 |
This unit test is not hitting the code in question - I need to have a second look .... |
@@ -155,6 +155,25 @@ function testExpectStream() { | |||
); | |||
} | |||
|
|||
function testEmptyProps() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rullzer this test should now reflect your description. But this does not trigger the infinite loop. Can I ask you to have a look what's missing? THX
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took a look but it indeed does not seem to be triggered anymore. Maybe there was a different bug. Anyway good it is fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe related to an older version. Let's close this for now - let us know if the issue pops up again. THX
615a77d
to
cd360d3
Compare
Follow up of #70