Skip to content

Conversation

@wzy1935
Copy link

@wzy1935 wzy1935 commented Apr 22, 2022

Description

Fixes the bug that after compression pdf becomes larger. Now the compressed file is always smaller than the original one, and the setting for the compression rate is more effective. No dependency changed.

Fixes #539

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Test compressing pdf with following three:

book_sample.pdf
pdf_with_big_pic.pdf
pdf_with_pic_hard_to_compress.pdf

  • ./gradlew assembleDebug assembleRelease
  • ./gradlew checkstyle

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 010e62e and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2

View more on Code Climate.

@wzy1935
Copy link
Author

wzy1935 commented Apr 22, 2022

@codegsaini Hi! I refactored it in bf63f0e. I think it can pass complexity analysis now?

Copy link
Contributor

@codegsaini codegsaini left a comment

Choose a reason for hiding this comment

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

This PR looks good. @Swati4star Can consider final review on it before merging.
Happy coding :)

@wzy1935
Copy link
Author

wzy1935 commented Apr 24, 2022

@codegsaini It should work now. Thank you for the code review!

Copy link
Owner

@Swati4star Swati4star left a comment

Choose a reason for hiding this comment

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

Can you fix the merge conflicts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The size of the compressed pdf becomes larger

3 participants