Skip to content
This repository has been archived by the owner on Dec 5, 2022. It is now read-only.

skip warning #67

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

skip warning #67

wants to merge 3 commits into from

Conversation

renatose
Copy link
Contributor

as mode_last == mode_loop, Mode::Loop == vlc::mode_last
this way the warning is skipped

as mode_last == mode_loop, Mode::Loop == vlc::mode_last
@RSATom
Copy link
Owner

RSATom commented Aug 31, 2016

it was maid so just to avoid possible problems if order of items in playback_mode_e will be changed by some reason. So it's better fix this warning by reinterpret_cast<playback_mode_e>(mode)

@RSATom
Copy link
Owner

RSATom commented Aug 31, 2016

btw, it worth add there check for values below 0 too since enums by default are signed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants