Console View
|
|
|
|||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
|
Remove goto-based error handling in pdffonts.cc Now that globalParams is a unique_ptr, there is no need for these gotos anymore. |
|||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
| Remove obsolete label 'err1' | |||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
| Make globalParams a std::unique_ptr | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Fix uninitialized memory use in JBIG2Stream::readTextRegionSeg When the unlikely condition triggers |
|||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
| Do not include string.h, it is not used | |||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
| Use std::unique_ptr for passwords | |||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
| Use std::unique_ptr for PDFDoc | |||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
|
Use std::string for the filename ... and create it on the stack rather than on the heap. Makes the code more readable. |
|||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
|
Remove the 'err0' error handling goto target It is used only once, and removing it allows me to move variable declarations around more freely. |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| CI: tweak the fedora builders | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
| CI: we can go back to current fedora for clazy-ci | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Update (C) of previous commits | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
| qt5: remove a bunch of unused internal functions | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Introduce Object::getNumWithDefaultValue Is like getNum but instead of asserting if Object is not a num it returns the given default value I find it much easier to read rect->x1 = obj1.arrayGet(0).getNumWithDefaultValue(0); than (obj2 = obj1.arrayGet(0), obj2.isNum() ? rect->x1 = obj2.getNum() : rect->x1 = 0); On top of it has the benefit of being slightly faster |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Use Fedora 30 for clazy while they fix Fedora 31 | |||
|
Adriaan de Groot
groot@kde.org |
|
||
| unicodeNormalizeNFKC: Make in const | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
| 0.82.0 | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Import md5 code from xpdf 4.02 Fixes issue 832 |
|||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
|
Make FontInfo::scan return a std::vector object ... rather than a pointer to a std::vector. Given that a std::vector is little more than a pointer and some size information, there is no need to create std::vector objects on the heap. Returning them by value is just as fast (the vector content is not copied), and makes the code more readable, too. |
|||
|
Oliver Sander
oliver.sander@tu-dresden.de |
|
||
|
Use a std::unique_ptr for XRef This makes memory handling a tiny bit more robust. |
|||
|
|||
|
Bohumir Zamecnik
bohumir.zamecnik@gmail.com |
|
||
|
Install Cairo headers if Cairo has been found. Closes #83. |
|||
|
Simon McVittie
smcv@debian.org |
|
||
|
Silence deprecation warnings for PopplerAttachment->ctime, ->mtime GTime is not Y2038-safe, and is now marked as deprecated. Don't trigger deprecation warnings for projects that include poppler headers but do not otherwise use GTime. Part of #765. Signed-off-by: Simon McVittie <smcv@debian.org> |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Update (C) | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
| CharCodeToUnicode::mapToUnicode: Make clear the data is const | |||
|
corentin
corentinf@free.fr |
|
||
|
Add const in CharCodeToUnicode Function updated: getLength mapToUnicode |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Fix assert on malformed documents | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Update (C) of previous commit | |||
|
Stefan Brüns
stefan.bruens@rwth-aachen.de |
|
||
|
Move the non-trivial part of the clip test to the implementation file This allows to only have a SplashXPathScanner forward declaration in the header file. |
|||
|
Stefan Brüns
stefan.bruens@rwth-aachen.de |
|
||
| Include SplashMath.h only where needed | |||
|
Even Rouault
even.rouault@spatialys.com |
|
||
|
Do not override user-defined CMAKE_C[XX]_FLAGS for clang 337585e3d881c2c2c9099888b09902119dc05bf8 unconditionnaly overrides any potential user-defined CMAKE_C[XX]_FLAGS. This for example breaks the GDAL oss-fuzz builds which build Poppler from source (see https://github.com/OSGeo/gdal/blob/master/gdal/fuzzers/build.sh#L54) So do the same as the GCC path where we save input CMAKE_C[XX]_FLAGS and reinject them in custom CMAKE_C{XX}_FLAGS_{build_configuration} The values are identical to GCC, execpt for the _DEBUG configuration where we remove '-O2 -fno-reorder-blocks -fno-schedule-insns -fno-inline' since clang does not support -fno-reorder-blocks and -fno-schedule-insns, so it is likely better to disable any optimization for proper debugging. |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Set our default cxx and c flags to clang too This code needs a bit of refactoring to share most of it between gcc and clang |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Enable a few more clang-tidy bugprone checks | |||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Enable clang-tidy bugprone-too-small-loop-variable And fixes for it in the code |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| GfxCIDFont::getCodeToGIDMap: n is const and an int | |||
|
Christian Persch
chpe@src.gnome.org |
|
||
|
glib: Add poppler_document_new_from_bytes This allows creating a PopplerDocument from data stored in a GBytes. This is better than poppler_document_new_from_data() since the document will keep a reference to the data, instead of having to make sure manually that the data stays around as long as the document exists. Also poppler_document_new_from_data() is buggy in that it uses int for the data size instead of gsize. https://gitlab.freedesktop.org/poppler/poppler/issues/771 |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Use range for loops to iterate const arrays for (const TypeMapEntry &entry : typeMap) { is much easier to read than for (unsigned i = 0; i < sizeof(typeMap) / sizeof(typeMap[0]); i++) { |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Run clang-tidy on CI Only with the performance- checks enabled for now |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Also switch the const_cast in Stream This way we only const_cast in free() |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
|
Move the const_cast from assignment to free The problem is that some of the times the pointers hold values to const tables and some others hold values to dynamic memory. Instead of const_casting the const tables when needed and holding a non const pointer, we hold a const pointer and only const cast on free if needed |
|||
|
Albert Astals Cid
aacid@kde.org |
|
||
| Update (C) from previous commit | |||
