[ 
https://issues.apache.org/jira/browse/PDFBOX-1684?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13900529#comment-13900529
 ] 

Fred Hansen commented on PDFBOX-1684:
-------------------------------------

BTW: Where does one send comments that need to be in the upgrade notice.  That 
is, the notice that tells programmers what to change in existing code in order 
to convert from 1.x to 2.0?

> DateConverter source is in three places and has diverged
> --------------------------------------------------------
>
>                 Key: PDFBOX-1684
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-1684
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: Utilities
>    Affects Versions: 2.0.0
>         Environment: Win 8, maven 3.0.5, svn 1.7.8 (r1419691), javac 1.7.0_25
>            Reporter: Fred Hansen
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> Versions of DateConverter,java are in three (3) packages: jempbox.impl, 
> pdfbox.util, and xmpbox. They have diverged as updates have been made to one 
> or the other. 
> Date Converter is used in these modules:
>     jempbox.xmp.ResourceEvent.java
>     jempbox.xmp.XMPSchema.java
>     pdfbox.cos.COSDictionary.java
>     pdfbox.pdmodel.fdf.FDFAnnotation.java
>     preflight.metadata.SynchronizedMetaDataValidation.java
>     xmpbox.type.DateType.java
> The smallest library is xmpbox, so let's consider it as a home for 
> DateConverter. Preflight already imports xmpbox. Pdfbox requires jempbox, so 
> importing xmpbox into jempbox will satisfy both.  Since xmpbox defines some 
> xmp features, it seems likely that jempbox could take further advantage of 
> xmpbox.
> { incorrect: Conclusion and recommendation:  have a single copy of 
> DateConverter in xmpbox. }
> Conclusion: create another top-level pdfbox package for DateConverter and 
> other utilities that do not depend on pdfbox classes.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to