Details
-
Type: Improvement
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 4.1
-
Fix Version/s: 5.0
-
Component/s: Documentation
-
Labels:None
-
Environment:-
Description
Note from ICEpdf user:
I was browsing http://sventon.icefaces.org/svn/showfile.svn?path=/icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java&name=repo
and i noticed some code duplication.
The method "alternative1()" and "getAdobeColor()" share the same algorithm; the only difference is the first return's the RGB triplet in it's arithmetic format (0.0 to 1.0) and the second in the digital format (0 to 255).
The methods "getAutoCadColor()" and "getGhostColor()" share the same algorithm too.
For simplicity sake, you would probably want to get rid of the duplicates (or at least document this fact).
I was browsing http://sventon.icefaces.org/svn/showfile.svn?path=/icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java&name=repo
and i noticed some code duplication.
The method "alternative1()" and "getAdobeColor()" share the same algorithm; the only difference is the first return's the RGB triplet in it's arithmetic format (0.0 to 1.0) and the second in the digital format (0 to 255).
The methods "getAutoCadColor()" and "getGhostColor()" share the same algorithm too.
For simplicity sake, you would probably want to get rid of the duplicates (or at least document this fact).
Activity
Tyler Johnson
created issue -
Patrick Corless
made changes -
Field | Original Value | New Value |
---|---|---|
Salesforce Case | [] | |
Fix Version/s | 4.3.3 [ 10333 ] |
Patrick Corless
made changes -
Salesforce Case | [] | |
Fix Version/s | 4.5 [ 10342 ] | |
Fix Version/s | 4.3.3 [ 10333 ] |
Patrick Corless
made changes -
Fix Version/s | 5.0 [ 10314 ] | |
Fix Version/s | 4.5 [ 10342 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #34193 | Tue Apr 02 13:35:16 MDT 2013 | patrick.corless | |
Files Changed | ||||
MODIFY
/icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java
|
Patrick Corless
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Patrick Corless
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Pretty easy update to close this one targeting 4.3.3.