-
Notifications
You must be signed in to change notification settings - Fork 110
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
video_set_marquee_int not interpreting _Enum type of VideoMarqueeOption.Position properly, resulting in crash #243
Comments
In addition to the incorrect Class Using |
Investigating the
both define the value as The issue is Python class There are several other It is unclear how to resolved this in the Another option might be to use the signed Simply making
|
None of the options mentioned above are feasible. Better would to select the unsigned Below is are the changes made in class
|
See pull request for |
Hi All,
Trying to implement text overlays in using player.video_set_marquee_int, but found the following fix me note:
Error Received :
Temporary Fix 1
I have no experience in ctype binding, so not sure how to implement the fix properly, but it looks like the class type _Enum of VideoMarqueeOption.Position isn't resolving to an INT type properly? When I use the native VLC integer assignments (found here) I'm able to set the overlay position as expected, for example, setting it display the text on the bottom:
Temporary Fix 2
Alternatively, if I set the enum entries without the Position() wrapper (not sure what to call this), it allows for using the enum and works:
Not sure why the other enums seem to work natively, but since I don't see any open issues discussing this, decided to start this thread with my findings.
Misc Comments
Additionally, I noticed two small discrepancies:
src block
img src
Hope this helps in troubleshooting.
The text was updated successfully, but these errors were encountered: