Skip to content
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

Jump to bottom

BR_GetMouseCursorContext_MIDI retval value #783

Closed
X-Raym opened this issue Dec 14, 2015 · 2 comments
Closed

BR_GetMouseCursorContext_MIDI retval value #783

X-Raym opened this issue Dec 14, 2015 · 2 comments
Labels
bug

Comments

@X-Raym
Copy link
Contributor

X-Raym commented Dec 14, 2015

Hi,
According to the doc and to the previous scripts we already made, BR_GetMouseCursorContext_MIDI should return
retval, inlineEditor, noteRow, ccLane, ccLaneVal, ccLaneId
It seems with REAPER 5.1 and SWS 2.8.2 that it now returns
inlineEditor, noteRow, ccLane, ccLaneVal, ccLaneId
and not matching with doc description.

More infos on this thread

Tested in Lua on Windows 10.

Thanks for your support !
@X-Raym X-Raym changed the title BR_GetMouseCursorContext_MIDI inverted variables BR_GetMouseCursorContext_MIDI retval value Dec 14, 2015
@X-Raym
Copy link
Contributor Author

X-Raym commented Dec 14, 2015

Maybe this is just an error in the doc.
I modify my scripts so that it handles this behavior.

Jeff0S added a commit that referenced this issue Jan 11, 2016
@Jeff0S
ReaScript: fix BR_GetMouseCursorContext_MIDI return value (issue #783)
655142a
@Jeff0S
Copy link
Member

Jeff0S commented Jan 11, 2016

Should be fixed for the next build, note that the return value will be back...
Thanks for the detailed bug report, it helped.

@Jeff0S Jeff0S closed this as completed Jan 11, 2016
@Jeff0S Jeff0S added the bug label Jan 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@X-Raym @Jeff0S