-
Notifications
You must be signed in to change notification settings - Fork 814
fix issue #539 #1054
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
base: master
Are you sure you want to change the base?
fix issue #539 #1054
Conversation
|
Code Climate has analyzed commit 010e62e and detected 2 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
|
@codegsaini Hi! I refactored it in bf63f0e. I think it can pass complexity analysis now? |
codegsaini
left a comment
There was a problem hiding this 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 :)
|
@codegsaini It should work now. Thank you for the code review! |
Swati4star
left a comment
There was a problem hiding this 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?
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
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 checkstyleChecklist: