Skip to content

Separate the tiff encoder classes - #1527

Merged
brianpopow merged 4 commits into
SixLabors:tiff-formatfrom
IldarKhayrutdinov:tiff-format-refactory
Feb 1, 2021
Merged

brianpopow merged 4 commits into
SixLabors:tiff-formatfrom
IldarKhayrutdinov:tiff-format-refactory

Conversation

@IldarKhayrutdinov

@IldarKhayrutdinov IldarKhayrutdinov commented Jan 31, 2021 •

Copy link
Copy Markdown
Contributor

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

  • Split TiffWriter class.
  • Move decompressor classes.

Only splitting and moving classes, no change algorithms.

@IldarKhayrutdinov IldarKhayrutdinov mentioned this pull request Jan 31, 2021
17 tasks done
@IldarKhayrutdinov
IldarKhayrutdinov marked this pull request as ready for review January 31, 2021 14:06

@brianpopow brianpopow left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍

/// <param name="memoryAllocator">The memory allocator.</param>
/// <param name="configuration">The configuration.</param>
/// <param name="entriesCollector">The entries collector.</param>
public TiffBaseColorWriter(TiffStreamWriter output, MemoryAllocator memoryAllocator, Configuration configuration, TiffEncoderEntriesCollector entriesCollector)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

protected?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, done!


public Stream BaseStream { get; }

protected MemoryAllocator MemoryAllocator { get; }

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MemoryAllocator and Configuration below seems not to be used. Can they be removed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, done!

@IldarKhayrutdinov IldarKhayrutdinov changed the title Tiff format refactory Separate the tiff encoder classes Feb 1, 2021
@brianpopow
brianpopow merged commit 1aea01a into SixLabors:tiff-format Feb 1, 2021
JimBobSquarePants pushed a commit that referenced this pull request Feb 17, 2021
JimBobSquarePants pushed a commit that referenced this pull request Feb 17, 2021
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