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

Preserve settings for builtin formats #301

Merged

Conversation

c72578
Copy link
Collaborator

@c72578 c72578 commented Oct 21, 2023

The settings of builtin formats were not disabled anymore
in Settings - Formats since e1f8906 (last working in release: 2.1.7).
They were only disabled at the first start of CUETools. As soon as a
settings.txt file was available, this was not the case anymore.

  • Furthermore, this fixes availability of newly added formats like
    aiff, if a settings.txt file from a previous version of CUETools is
    already present.

@c72578

This comment was marked as outdated.

@ha-korth
Copy link

ha-korth commented Oct 21, 2023

Locking Encoders, Decoders, and Advanced tabs now.
No new encoders/decoders saved on restart.
Encoder/decoder setting changes revert back to default on restart.
Advanced options changes revert back to default on restart.

@c72578 c72578 marked this pull request as draft October 21, 2023 12:34
The settings of builtin formats were not disabled anymore
in Settings - Formats since e1f8906 (last working in release: 2.1.7).
They were only disabled at the first start of CUETools. As soon as a
settings.txt file was available, this was not the case anymore.

- Furthermore, this fixes availability of newly added formats like
  aiff, if a settings.txt file from a previous version of CUETools is
  already present.
@c72578 c72578 force-pushed the 2023-10-21_Preserve_settings_for_builtin_formats branch from 71488ef to 6eae07c Compare October 21, 2023 14:30
@c72578
Copy link
Collaborator Author

c72578 commented Oct 21, 2023

@ha-korth thanks for testing and the feedback.
The PR has been updated. Here is a new build for testing:
CUETools_2.2.4_2023-10-21_git_6eae07c_Preserve_settings_for_builtin_formats.zip
SHA256: a5ef5b29da316c75cb90dd66f6e7be758f68c2f78def5b297fb6350aa533f6a1

@c72578 c72578 marked this pull request as ready for review October 21, 2023 15:09
@ha-korth
Copy link

Initial run-through looks good.
Previous settings/changes preserved.
Encoders, Decoders, and Advanced tabs okay.
Formats tab okay.
Profile test looks good also.

@c72578 c72578 merged commit 6eae07c into gchudov:master Oct 26, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants