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.

Sign up for GitHub

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

Jump to bottom

In sequence.c, start/end parameter check is corrected for arrays #297

Open

k-okada wants to merge 1 commit into euslisp:master from k-okada:fix_sequence

Member

k-okada commented Jul 17, 2018

   since specifying start>end caused a segmentation fault.

This PR was once merged in #285, but reverted #293
because this causes huge regression for roseus jsk-ros-pkg/jsk_roseus#568


          In sequence.c, start/end parameter check is corrected for arrays

bc48fb4

       since specifying start>end caused a segmentation fault.

k-okada added the PR/WaitForFeedback label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR/WaitForFeedback