> Your patch is making some of the time module tests to fail:
     > 
     > 
base/time/pdf-time-span-add.c:109:F:pdf_time_span_add:pdf_time_span_add_002:0: 
Failure 'status != PDF_ERROR' occured
     > 
base/time/pdf-time-span-copy.c:93:F:pdf_time_span_copy:pdf_time_span_copy_002:0:
 Failure 'status != PDF_ERROR' occured
     > 
base/time/pdf-time-span-diff.c:105:F:pdf_time_span_diff:pdf_time_span_diff_002:0:Failure
 'status != PDF_ERROR' occured
     > 
    
I fixed the time api, implementation and tests to expect/return
PDF_EBADDATA in those functions.  Then I applied your patch.

AFAIK no more API changes are needed.

--
Jose E. Marchesi    [email protected]
GNU Project         http://www.gnu.org




Reply via email to