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.

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

player: don't load encoding mode profile twice #15227

Merged
merged 2 commits into from
Oct 31, 2024
Merged

player: don't load encoding mode profile twice #15227

merged 2 commits into from
Oct 31, 2024

Commits on Oct 30, 2024

  1. options: remove stale comment about encoding mode being compiled-in 

    Encoding mode used to be a compile time option, but now it's always
compiled in.
    @llyyr
    llyyr committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    0a2f757 View commit details
    Browse the repository at this point in the history

  2. player: don't load encoding mode profile twice 

    Also move the loading of profiles and input section activation in one
place. Leaving this in mp_initialize has the drawback that encoding
section profile will overwrite options set by the user in the command
line, since it's set after the CLI options are parsed and processed.
mp_parse_cfgfiles happens before parsing the CLI, so loading the profile
there is better.
    @llyyr
    llyyr committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    9944a81 View commit details
    Browse the repository at this point in the history