OpenSystemsDevelopment / qpdf

Commit bd8918fffc73803698f02021543a14445ba17e0b

Authored by Jay Berkenbilt
1 parent 848351f1

Fix up TODO for ABI changes

Inline Side-by-side
Showing 1 changed file with 33 additions and 17 deletions
1 Next ABI 1 Next ABI
2 ======== 2 ========
3 3
4 -Do these things next time we have to break binary compatibility 4 + * Rename QUtil::strcasecmp since strcasecmp is a macro on some
  5 + platforms. See #242.
  6 +
  7 + * Get rid of the version of QPDF::copyForeignObject with the bool
  8 + unused parameter.
  9 +
  10 + * Remove version of QPDFTokenizer::expectInlineImage with no
  11 + arguments.
  12 +
  13 + * Build with -fvisibility=hidden by default. Fix QPDF_DLL. See #302
  14 + for discussion. See also https://gcc.gnu.org/wiki/Visibility
5 15
6 * Check all classes that don't use the Members pattern and see if 16 * Check all classes that don't use the Members pattern and see if
7 they should be converted. Most of the pipeline classes should be 17 they should be converted. Most of the pipeline classes should be
8 converted. 18 converted.
9 19
  20 + Buffer.hh
  21 + BufferInputSource.hh
  22 + FileInputSource.hh
  23 + InputSource.hh
  24 + Pl_Buffer.hh
  25 + Pl_Concatenate.hh
  26 + Pl_Count.hh
  27 + Pl_DCT.hh
  28 + Pl_Discard.hh
  29 + Pl_Flate.hh
  30 + Pl_RunLength.hh
  31 + Pl_StdioFile.hh
  32 + QPDFExc.hh
  33 + QPDFObjGen.hh
  34 + QPDFSystemError.hh
  35 + QPDFXRefEntry.hh
  36 +
10 * Pl_Buffer's internal structure is not right for what it does. It 37 * Pl_Buffer's internal structure is not right for what it does. It
11 was modified for greater efficiency, but it was done in a way that 38 was modified for greater efficiency, but it was done in a way that
12 preserved binary compatibility, so the implementation is a bit 39 preserved binary compatibility, so the implementation is a bit
13 convoluted. 40 convoluted.
14 41
15 - * Rename QUtil::strcasecmp since strcasecmp is a macro on some  
16 - platforms. See #242.  
17 -  
18 * Check all overloaded methods to see if any can be eliminated by 42 * Check all overloaded methods to see if any can be eliminated by
19 using defaulted arguments. See ../misc/find-overloaded-functions.pl 43 using defaulted arguments. See ../misc/find-overloaded-functions.pl
20 (not in source repo) 44 (not in source repo)
21 45
22 - * Get rid of the version of QPDF::copyForeignObject with the bool  
23 - unused parameter.  
24 -  
25 - * Remove version of QPDFTokenizer::expectInlineImage with no  
26 - arguments.  
27 -  
28 - * Consider whether to build with -fvisibility=hidden by default. See  
29 - #302 for discussion.  
30 -  
31 - * See if there is a way to provide an option to use higher  
32 - compression with zlib. See #113. This can be done by adding options  
33 - to Pl_Flate and having QPDFWriter optionally use those options.  
34 -  
35 Lexical 46 Lexical
36 ======= 47 =======
37 48
@@ -120,6 +131,11 @@ directory or that are otherwise not publicly accessible. This includes @@ -120,6 +131,11 @@ directory or that are otherwise not publicly accessible. This includes
120 things sent to me by email that are specifically not public. Even so, 131 things sent to me by email that are specifically not public. Even so,
121 I find it useful to make reference to them in this list 132 I find it useful to make reference to them in this list
122 133
  134 + * See if there is a way to provide an option to use higher
  135 + compression with zlib. See #113. This can be done by adding options
  136 + to Pl_Flate and having QPDFWriter optionally use those options.
  137 + Also increasing the buffer size may help.
  138 +
123 * Add support for writing name and number trees 139 * Add support for writing name and number trees
124 140
125 * Figure out how to render Gajić correctly in the PDF version of the 141 * Figure out how to render Gajić correctly in the PDF version of the