Browse Source

Nicer initialization of the options.

af/merge-core
Dirk Lemstra 9 years ago
committed by Dirk Lemstra
parent
commit
01d2f42cdd
  1. 9
      src/ImageSharp.Formats.Gif/GifDecoderOptions.cs
  2. 9
      src/ImageSharp.Formats.Png/PngDecoderOptions.cs
  3. 15
      src/ImageSharp/Formats/DecoderOptions.cs
  4. 15
      src/ImageSharp/Formats/EncoderOptions.cs

9
src/ImageSharp.Formats.Gif/GifDecoderOptions.cs

@ -19,7 +19,6 @@ namespace ImageSharp.Formats
/// </summary>
public GifDecoderOptions()
{
this.InitializeWithDefaults();
}
/// <summary>
@ -29,13 +28,12 @@ namespace ImageSharp.Formats
private GifDecoderOptions(IDecoderOptions options)
: base(options)
{
this.InitializeWithDefaults();
}
/// <summary>
/// Gets or sets the encoding that should be used when reading comments.
/// </summary>
public Encoding TextEncoding { get; set; }
public Encoding TextEncoding { get; set; } = DefaultEncoding;
/// <summary>
/// Converts the options to a <see cref="GifDecoderOptions"/> instance with a cast
@ -53,10 +51,5 @@ namespace ImageSharp.Formats
return new GifDecoderOptions(options);
}
private void InitializeWithDefaults()
{
this.TextEncoding = DefaultEncoding;
}
}
}

9
src/ImageSharp.Formats.Png/PngDecoderOptions.cs

@ -19,7 +19,6 @@ namespace ImageSharp.Formats
/// </summary>
public PngDecoderOptions()
{
this.InitializeWithDefaults();
}
/// <summary>
@ -29,13 +28,12 @@ namespace ImageSharp.Formats
private PngDecoderOptions(IDecoderOptions options)
: base(options)
{
this.InitializeWithDefaults();
}
/// <summary>
/// Gets or sets the encoding that should be used when reading text chunks.
/// </summary>
public Encoding TextEncoding { get; set; }
public Encoding TextEncoding { get; set; } = DefaultEncoding;
/// <summary>
/// Converts the options to a <see cref="IPngDecoderOptions"/> instance with a cast
@ -53,10 +51,5 @@ namespace ImageSharp.Formats
return new PngDecoderOptions(options);
}
private void InitializeWithDefaults()
{
this.TextEncoding = DefaultEncoding;
}
}
}

15
src/ImageSharp/Formats/DecoderOptions.cs

@ -15,7 +15,6 @@ namespace ImageSharp
/// </summary>
public DecoderOptions()
{
this.InitializeWithDefaults();
}
/// <summary>
@ -24,23 +23,15 @@ namespace ImageSharp
/// <param name="options">The decoder options</param>
protected DecoderOptions(IDecoderOptions options)
{
if (options == null)
if (options != null)
{
this.InitializeWithDefaults();
return;
this.IgnoreMetadata = options.IgnoreMetadata;
}
this.IgnoreMetadata = options.IgnoreMetadata;
}
/// <summary>
/// Gets or sets a value indicating whether the metadata should be ignored when the image is being decoded.
/// </summary>
public bool IgnoreMetadata { get; set; }
private void InitializeWithDefaults()
{
this.IgnoreMetadata = false;
}
public bool IgnoreMetadata { get; set; } = false;
}
}

15
src/ImageSharp/Formats/EncoderOptions.cs

@ -15,7 +15,6 @@ namespace ImageSharp
/// </summary>
public EncoderOptions()
{
this.InitializeWithDefaults();
}
/// <summary>
@ -24,23 +23,15 @@ namespace ImageSharp
/// <param name="options">The encoder options</param>
protected EncoderOptions(IEncoderOptions options)
{
if (options == null)
if (options != null)
{
this.InitializeWithDefaults();
return;
this.IgnoreMetadata = options.IgnoreMetadata;
}
this.IgnoreMetadata = options.IgnoreMetadata;
}
/// <summary>
/// Gets or sets a value indicating whether the metadata should be ignored when the image is being encoded.
/// </summary>
public bool IgnoreMetadata { get; set; }
private void InitializeWithDefaults()
{
this.IgnoreMetadata = false;
}
public bool IgnoreMetadata { get; set; } = false;
}
}

Loading…
Cancel
Save