summaryrefslogtreecommitdiff
path: root/writerfilter/qa
AgeCommit message (Collapse)AuthorFilesLines
2016-03-11writerfilter: avoid ugly cross-directory includeMiklos Vajna1-20/+20
Change-Id: I16db4e16e5c2fa3aa3bfb22f83d2c36695499b5c
2016-03-10Extract Directories from BootstrapFixtureBaseStephan Bergmann1-1/+1
(as some tests derive from the latter only for the Directories part, not for the setUp/tearDown overrides: those tests will be cleaned up next) Change-Id: Ib6b78eea868b8bc21d4cc6e8fd9e1d025deca05f Reviewed-on: https://gerrit.libreoffice.org/23078 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-08tdf#97703 Removed empty setUp/tearDown methodsChirag Manwani1-11/+0
Change-Id: Ia62de30ae94bdae87cd5109c44eab40af4d020e5 Reviewed-on: https://gerrit.libreoffice.org/22985 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-07improve defaultparams lopluginNoel Grandin1-2/+1
to catch calling params with defaults like "= OUSString()" Change-Id: Iad060e318ed492c22f8be44e326174fe6d28fff9 Reviewed-on: https://gerrit.libreoffice.org/22932 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-01-31boost::tuple -> std::tupleCaolán McNamara1-71/+70
Change-Id: Ia08c184b792ff73d3ed49d6a5d54dec1a35943e9 Reviewed-on: https://gerrit.libreoffice.org/21955 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-12-08writerfilter: loplugin:stringconstantMiklos Vajna1-2/+2
Change-Id: Idb0865f4d0db6c4eda33810ab2ef4cff49bc85d1
2015-12-08tdf#54584: adjust skip-slash condition, allow unterminated quoteMike Kaganski1-0/+20
Experimenting with different non-alpha characters in front of field, most of them aren't allowed and result in field not recognized by MS Word: #$%&'()*+,-./:;<>?@[]^_`{|}~ Besides, if backslash "\" is followed by another backslash or space, it is illegal, too. This patch takes care of it. On the other side, not closing quotes is allowed by MS Word. This patch allows this, too. The patch does not handle another allowed field code "=2+2". This should be done in another commit. Change-Id: I842fe59c026b68977e61a7ae0b5495c02803ad83 Reviewed-on: https://gerrit.libreoffice.org/20435 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-11-05use uno::Reference::set method instead of assignmentNoel Grandin1-1/+1
Change-Id: I31a69a997098eb1807361b8049c3312a4f287d75
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2-4/+4
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-05-26need DISABLE_CVE_TESTS for these tests under windows tooCaolán McNamara1-0/+2
Change-Id: I92de8fa6a48dac9a0a09e6ebda4af9b8e4c3a1d7
2015-04-20sw, writerfilter: these 'using namespace' statements can be avoided easilyMiklos Vajna1-3/+1
Especially the 'using namespace std' is scary, now that we have std::shared_ptr and boost::shared_ptr, too. Change-Id: Ibb584281f1b9d56103ab5984473eb484157c12d5
2015-03-19convert SFX_FILTER_ constants to enum classNoel Grandin1-2/+2
Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613
2015-03-18create new 'enum class' SotClipboardFormatId to unify typesNoel Grandin1-2/+2
of which there are several. There are some issues here I am unsure of - the SW and SC and CHART2 modules essentially ignore the enum values and assign their own ids Perhaps I should change them to use the common values and create new enum values where necessary? - the sc/qa/ and sq/qa/ and starmath/qa/ code was doing some dodgy stuff. I translated the code to pass down the stuff numeric values to the underlying code, but perhaps further fixing is necessary? Change-Id: Ic06d723e404481e3f1bca67c43b70321b764d923
2015-01-11Use std::initializer_list ctorMiklos Vajna1-3/+4
Change-Id: I0238d8f4ae6b92f92f8bb92df33da5552a158cbe
2014-10-22i#84172 RTF import: handle nested groups inside redlinesMiklos Vajna1-0/+11
The problem was that in case we had {\revised foo {bar}}, then we only started a redline before "foo", but we ended it twice: during handling of both "}". Make sure that the end of the redline is only handled by the second one. Change-Id: I41ab8b486e8615d3479fa0fadd6016476859b4b2
2014-10-18i#74153 RTF import: fix track changes vs superscript char propMiklos Vajna1-0/+8
Since 9dbf817fe5c5253fba0831aefa17575ae0ba3af1 (handle scope of w:pPrChange and w:rPrChange properly (bnc#821804), 2014-10-01), this caused an assertion failure -> crash. Change-Id: I680edf58854d0c9be6e77c44c1e6beb1e0442a49
2014-10-06encrypt sf_508f4e1..rtf to avoid it triggering virus detectionCaolán McNamara2-0/+0
use name CVE-pseudo-2009-0238-1.rtf to record its detected as Virus:Trojan.Mdropper and pseudo because its just a fuzzed file from some random .rtf and not an original reproducer for CVE-2009-0238 Change-Id: I85f6a72e0d32c14ea27cbe03bc4da3c2306fd1f5
2014-08-21NS_ooxml::LN_CT_LevelText_val might be missingCaolán McNamara1-0/+57
Change-Id: I2683dd30d521b91cf026b3553ed4f75ee4172b3a
2014-08-20empty ReferenceCaolán McNamara1-0/+62
valgrind + bff Change-Id: Id3c039a46dec5d2d4a4642dfb53d23a76972dde2
2014-08-20empty m_aStatesCaolán McNamara1-0/+62
valgrind + bff Change-Id: I3bd526b7ebf0f4fce5d0c7054809e0dc2908e73f
2014-08-19empty m_aStatesCaolán McNamara1-0/+0
valgrind + bff Change-Id: I7b45e10471de77bd0481adf0bccdbf852c36d815
2014-08-18check for empty m_aStatesCaolán McNamara1-0/+0
valgrind + bff Change-Id: Ib818d508e10877a845b733c4aa223d1e2cbbc05e
2014-07-28writerfilter: nobody reads the classfordefine attributeMiklos Vajna1-0/+3
Change-Id: I030137cb379de04968fb144516e90dbd1bdd97f6
2014-07-28writerfilter: add script to detect generated code changesMiklos Vajna1-0/+47
Change-Id: I340ef55dc9a09bd196469ed51848b22ba986de8c
2014-07-15fdo#76803: Kill resourcemodel::Fraction, and use Fraction from tools instead.Jan Holesovsky1-17/+0
Change-Id: I72d336b8aacf80f9ab6472c0948766ee56fda34f
2014-07-14writerfilter: add unit test for Fraction initializationMichael Stahl1-1/+16
... as used by wrap polygon import, see fdo#41068 fdo#76803. Change-Id: I4467560c2c3bbdab3fc9f8cd7a0c0b415429186c
2014-07-11new loplugin: externalandnotdefinedNoel Grandin1-13/+2
Find "missing headers," where a function is declared directly in the .cxx (as extern) and not defined, and should arguably instead be declared in an include file. Change-Id: I6d83ee432b2ab0cd050aec2b27c3658d32ac02a2
2014-07-09unit test for convertTwipToMM100 functions (cp#1000087)Matúš Kukan1-1/+26
Change-Id: I36d68ecaa71319d90b9abaec21543cc5d4e76f24
2014-04-22Resolves: rhbz#1089377 crash on loading a specific rtfCaolán McNamara1-0/+1719
Change-Id: I26864f3a2247f5d7184f395cfed91eaf92e36295
2014-04-11writerfilter: remove no longer needed WRITERFILTER_WRITERFILTER_DLLPUBLICMiklos Vajna1-1/+0
Change-Id: Idedee38be19bc770518b85af62c9fc2b5b64e822
2014-04-11test cases for CVE-2014-1761Caolán McNamara2-0/+0
Change-Id: I8fa160818e5981090aadea7d4691b8acc61f0b40
2014-03-27Second batch of adding SAL_OVERRIDE to overriding function declarationsStephan Bergmann2-4/+4
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: Ie656f9d653fc716f72ac175925272696d509038f
2014-03-03writerfilter: salvage a field parameter parsing train wreckMichael Stahl1-0/+162
Field parameters get horribly maimed by lcl_ExtractParameter which clearly has never worked in its 7 years of existence (and looking at the inanity at the call sites makes one wonder what the author was smoking). The format is actually quite annoying, since spaces between parameters are optional. The old RTF filter was at least able to parse "PAGEREF bookmark" fields, so this fixes such regressions (related: rhbz#1065629). Change-Id: I9b2e32c3c7264be0fc1077cb8fb3f1bc5c1955bb
2014-02-17abi#3623 RTF import: missing null pointer checkMiklos Vajna1-0/+7
Change-Id: I5861e5fec0d822f42dfeb994500a818a968e69fb
2014-02-06writerfilter: whitespace fixes in testrtftokMiklos Vajna1-8/+8
Change-Id: I520595debaa47cd38be88428480ff2de9d7bf1b6
2013-11-04fdo#49666 RTF import: fix crash on picture frame with shape textMiklos Vajna1-0/+13
The bugdoc contains a shape with type 75 (ESCHER_ShpInst_PictureFrame), which also has a shape text. Looks like this is an RTF which is generated by some 3rd-party tool: Word can open it, but it doesn't show the text on the picture frame, either. So, let's just ignore the shape text for picture frames at RTF import time as well, that avoids the crash. Change-Id: If673122eb16f4a4f7eddf107877fcfa7cb052821
2013-09-23writerfilter: remove dead doctok unit testMiklos Vajna3-264/+0
Change-Id: I4d223611df92a7e348dc31e38bd45849fe0eb028
2013-09-10writerfilter: remove old unit testsJelle van der Waa8-1267/+0
Change-Id: Icaa0560b812b96e41c52d7a43267873fe364ceb7 Reviewed-on: https://gerrit.libreoffice.org/5904 Reviewed-by: Andrzej J.R. Hunt <andrzej@ahunt.org> Tested-by: Andrzej J.R. Hunt <andrzej@ahunt.org>
2013-09-10remove old unit testsJelle van der Waa2-7552/+0
Change-Id: I83619c47d435b9ee7bd0de3121832f84046a6d18 Reviewed-on: https://gerrit.libreoffice.org/5903 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2013-07-10writerfilter: move the crash test file to "pass" directoryMichael Stahl1-0/+12
Change-Id: I8a97eac789e3e8234673de87653b78c39dafe14c
2013-06-11abi#4817 attempt to access an element in an empty containerMiklos Vajna1-0/+6
Change-Id: Ic51ef8b3db182dd34195c9bb808170d20953d406
2013-06-10fdo#64656 RTF import: handle levelnumbers without leveltextMiklos Vajna1-0/+10
Change-Id: I34ce7b172dc94489f7f506796e66b14598aab19c
2013-05-21fdo#46808, Use service constructorsNoel Grandin1-14/+4
Change-Id: Ib428bb78751e9707c89dcf311b8ab86ad8184775
2013-05-02writerfilter: few more doxygen commentsMiklos Vajna1-0/+7
Change-Id: I4893e2b3297adadf4e19ab22b298aaef4044b52e
2013-04-22Move to MPLv2 license headers, with ESC decision and author's permission.Michael Meeks1-23/+4
2013-04-07mass removal of rtl:: prefixes for O(U)String*Luboš Luňák2-8/+8
Modules sal, salhelper, cppu, cppuhelper, codemaker (selectively) and odk have kept them, in order not to break external API (the automatic using declaration is LO-internal). Change-Id: I588fc9e0c45b914f824f91c0376980621d730f09
2013-03-19further OUString cleanupThomas Arnhold1-1/+1
Change-Id: I19954ee5976cb881a5fe73a889c8e04a19b14eb6
2013-02-25drop various dangling makefile.mksCaolán McNamara3-184/+0
Change-Id: Id76b57a1ef2edbb05f1d22a624caad077275ac74
2013-02-24remove RTL_CONSTASCII_(U)STRINGPARAMChr. Rossmanith1-4/+2
Change-Id: Ica4dc859229c2ba0dc052a97ff23178895c25580 Reviewed-on: https://gerrit.libreoffice.org/2368 Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
2013-01-03writerfilter: remove Junit complex testMichael Stahl6-136/+0
The test just loads 4 documents and has been disabled for months. Also remove the test documents because they are duplicates and are also stored in writerfilter/qa/documents/. Change-Id: I9f51240483d800bc104c3842b8158dec90b3e69b