Reset encoding mode to default for codecs with dynamic modes list,

when settings become incompatible with the chosen mode.
E.g. for flac if -11 was selected and nonsubset modes are disabled,
-7 is chosen instead.
This commit is contained in:
Grigory Chudov
2013-04-08 23:11:03 -04:00
parent d1e70fc7f3
commit e3d0c595c0
12 changed files with 86 additions and 36 deletions

View File

@@ -996,11 +996,8 @@ namespace CUERipper
private void resetEncoderModes(CUEToolsUDC encoder)
{
if (encoder.settings != null)
{
encoder.settings.PCM = AudioPCMConfig.RedBook;
buttonEncoderSettings.Enabled = encoder.settings.HasBrowsableAttributes();
}
if (encoder.settings != null) encoder.settings.PCM = AudioPCMConfig.RedBook;
buttonEncoderSettings.Enabled = encoder.settings != null && encoder.settings.HasBrowsableAttributes();
string[] modes = encoder.SupportedModes;
if (modes == null || modes.Length < 2)
{
@@ -1011,8 +1008,14 @@ namespace CUERipper
}
else
{
if (encoder.EncoderModeIndex == -1 && encoder.settings != null)
{
string defaultMode;
encoder.settings.GetSupportedModes(out defaultMode);
encoder.EncoderMode = defaultMode;
}
trackBarEncoderMode.Maximum = modes.Length - 1;
trackBarEncoderMode.Value = encoder.DefaultModeIndex == -1 ? modes.Length - 1 : encoder.DefaultModeIndex;
trackBarEncoderMode.Value = encoder.EncoderModeIndex == -1 ? modes.Length - 1 : encoder.EncoderModeIndex;
labelEncoderMode.Text = encoder.EncoderMode;
labelEncoderMinMode.Text = modes[0];
labelEncoderMaxMode.Text = modes[modes.Length - 1];

View File

@@ -34,12 +34,13 @@ namespace CUETools.Codecs.FLACCL
public class FLACCLWriterSettings : AudioEncoderSettings
{
public FLACCLWriterSettings()
: base("", "8")
: base()
{
}
public override string GetSupportedModes()
public override string GetSupportedModes(out string defaultMode)
{
defaultMode = "8";
return this.AllowNonSubset ? "0 1 2 3 4 5 6 7 8 9 10 11" : "0 1 2 3 4 5 6 7 8";
}

View File

@@ -37,12 +37,13 @@ namespace CUETools.Codecs.FLAKE
public class FlakeWriterSettings : AudioEncoderSettings
{
public FlakeWriterSettings()
: base("", "7")
: base()
{
}
public override string GetSupportedModes()
public override string GetSupportedModes(out string defaultMode)
{
defaultMode = "7";
return this.AllowNonSubset ? "0 1 2 3 4 5 6 7 8 9 10 11" : "0 1 2 3 4 5 6 7 8";
}

View File

@@ -188,9 +188,11 @@ namespace CUETools.Codecs.WMA
return formats;
}
public override string GetSupportedModes()
public override string GetSupportedModes(out string defaultMode)
{
return string.Join(" ", GetFormats(null).ConvertAll(s => s.modeName).ToArray());
var fmts = GetFormats(null);
defaultMode = fmts.Count > 0 ? fmts[fmts.Count - 1].modeName : "";
return string.Join(" ", fmts.ConvertAll(s => s.modeName).ToArray());
}
}

View File

@@ -8,7 +8,7 @@ namespace CUETools.Codecs
/// </summary>
/// <remarks>
/// When plugins with classes that provide <see cref="IAudioDest" /> are
/// registered, their <see cref="AudioEncoderClass" /> attributes are read.
/// registered, their <see cref="AudioEncoderClassAttribute" /> attributes are read.
/// </remarks>
/// <example>
/// <code lang="C#">using CUETools.Codecs;
@@ -19,7 +19,7 @@ namespace CUETools.Codecs
///}</code>
/// </example>
[AttributeUsage(AttributeTargets.Class, AllowMultiple = true)]
public sealed class AudioEncoderClass : Attribute
public sealed class AudioEncoderClassAttribute : Attribute
{
private string _encoderName, _extension;
private bool _lossless;
@@ -51,7 +51,7 @@ namespace CUETools.Codecs
get { return _settings; }
}
public AudioEncoderClass(string encoderName, string extension, bool lossless, int priority, Type settings)
public AudioEncoderClassAttribute(string encoderName, string extension, bool lossless, int priority, Type settings)
{
_encoderName = encoderName;
_extension = extension;

View File

@@ -25,13 +25,17 @@ namespace CUETools.Codecs
foreach (PropertyDescriptor property in TypeDescriptor.GetProperties(this))
property.ResetValue(this);
this.m_supported_modes = supported_modes;
this.EncoderMode = default_mode;
this.m_default_mode = default_mode;
//GetSupportedModes(out m_default_mode);
this.EncoderMode = m_default_mode;
}
private string m_supported_modes;
protected string m_supported_modes;
protected string m_default_mode;
public virtual string GetSupportedModes()
public virtual string GetSupportedModes(out string defaultMode)
{
defaultMode = m_default_mode;
return this.m_supported_modes;
}
@@ -101,23 +105,14 @@ namespace CUETools.Codecs
set;
}
[XmlIgnore]
[Browsable(false)]
public string[] SupportedModes
{
get
{
return this.GetSupportedModes().Split(' ');
}
}
[XmlIgnore]
[Browsable(false)]
public int EncoderModeIndex
{
get
{
string[] modes = this.SupportedModes;
string defaultMode;
string[] modes = this.GetSupportedModes(out defaultMode).Split(' ');
if (modes == null || modes.Length < 1)
return -1;
for (int i = 0; i < modes.Length; i++)
@@ -128,7 +123,8 @@ namespace CUETools.Codecs
set
{
string[] modes = this.SupportedModes;
string defaultMode;
string[] modes = this.GetSupportedModes(out defaultMode).Split(' ');
if (modes.Length == 0 && value < 0)
return;
if (value < 0 || value >= modes.Length)

View File

@@ -90,6 +90,7 @@
<Compile Include="PlaybackState.cs" />
<Compile Include="Properties\AssemblyInfo.cs" />
<Compile Include="SilenceGenerator.cs" />
<Compile Include="UserDefinedEncoderSettings.cs" />
<Compile Include="UserDefinedReader.cs" />
<Compile Include="UserDefinedWriter.cs" />
<Compile Include="WAVReader.cs" />

View File

@@ -0,0 +1,39 @@
using System;
using System.Collections.Generic;
using System.Text;
using System.ComponentModel;
namespace CUETools.Codecs
{
public class UserDefinedEncoderSettings : AudioEncoderSettings
{
public UserDefinedEncoderSettings()
: base()
{
}
public string SupportedModes
{
get
{
return m_supported_modes;
}
set
{
m_supported_modes = value;
}
}
public string DefaultMode
{
get
{
return m_default_mode;
}
set
{
m_default_mode = value;
}
}
}
}

View File

@@ -153,7 +153,7 @@ namespace CUETools.Processor
encoders = new CUEToolsUDCList();
foreach (Type type in CUEProcessorPlugins.encs)
foreach (AudioEncoderClass enc in Attribute.GetCustomAttributes(type, typeof(AudioEncoderClass)))
foreach (AudioEncoderClassAttribute enc in Attribute.GetCustomAttributes(type, typeof(AudioEncoderClassAttribute)))
{
try
{

View File

@@ -70,7 +70,7 @@ namespace CUETools.Processor
decs.Add(type);
}
//if (type.IsClass && !type.IsAbstract && typeof(IAudioDest).IsAssignableFrom(type))
if (Attribute.GetCustomAttributes(type, typeof(AudioEncoderClass)).Length > 0)
if (Attribute.GetCustomAttributes(type, typeof(AudioEncoderClassAttribute)).Length > 0)
{
encs.Add(type);
}

View File

@@ -43,7 +43,7 @@ namespace CUETools.Processor
type = null;
}
public CUEToolsUDC(AudioEncoderClass enc, Type enctype)
public CUEToolsUDC(AudioEncoderClassAttribute enc, Type enctype)
{
name = enc.EncoderName;
extension = enc.Extension;
@@ -118,7 +118,8 @@ namespace CUETools.Processor
{
get
{
return this.settings == null ? this.supported_modes : this.settings.GetSupportedModes();
string defaultMode;
return this.settings == null ? this.supported_modes : this.settings.GetSupportedModes(out defaultMode);
}
set
{
@@ -149,7 +150,7 @@ namespace CUETools.Processor
}
}
public int DefaultModeIndex
public int EncoderModeIndex
{
get
{

View File

@@ -2465,8 +2465,14 @@ namespace JDP
}
else
{
if (encoder.EncoderModeIndex == -1 && encoder.settings != null)
{
string defaultMode;
encoder.settings.GetSupportedModes(out defaultMode);
encoder.EncoderMode = defaultMode;
}
trackBarEncoderMode.Maximum = modes.Length - 1;
trackBarEncoderMode.Value = encoder.DefaultModeIndex == -1 ? modes.Length - 1 : encoder.DefaultModeIndex;
trackBarEncoderMode.Value = encoder.EncoderModeIndex == -1 ? modes.Length - 1 : encoder.EncoderModeIndex;
labelEncoderMode.Text = encoder.EncoderMode;
labelEncoderMinMode.Text = modes[0];
labelEncoderMaxMode.Text = modes[modes.Length - 1];