summaryrefslogtreecommitdiff
path: root/starmath
AgeCommit message (Collapse)AuthorFilesLines
2016-06-23tdf#97527 - vcl: reference-count MenuNoel Grandin1-3/+4
some places are marked with "dodgy"- need to check those to see what is going on, because they are leaving dangling pointers behind in the Menu class Change-Id: I41d5c7c0fec2f70ce9e3ffdc48cd03d26c0a869b Reviewed-on: https://gerrit.libreoffice.org/26516 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-23starmath: Add a test case for parsing -Takeshi Abe2-0/+98
Change-Id: I0fc0b7e2abbadee2f13d477c5ddd2290f37487a0 Reviewed-on: https://gerrit.libreoffice.org/26563 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-22Drop unused #includeTakeshi Abe3-29/+0
Change-Id: Ic0cf4605cc7d02fdd537d05cbaa744c52126e6f1 Reviewed-on: https://gerrit.libreoffice.org/26545 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-21This can be an assert()Takeshi Abe1-25/+22
Change-Id: I2b0a8f50359e5b12fa27bf48c355f736e2b05033 Reviewed-on: https://gerrit.libreoffice.org/26491 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-20starmath: Possible cases are exhaustedTakeshi Abe1-1/+1
Change-Id: Icc48aa8ffa6a56f804c4d01f2fb3bcdf7de838d4 Reviewed-on: https://gerrit.libreoffice.org/26397 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-16starmath: more assertions holdTakeshi Abe1-21/+20
Change-Id: I9a7fb2f4d45c5b34c0ea1d186dd90353fedc8a6d Reviewed-on: https://gerrit.libreoffice.org/26299 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-14tdf#94205: Some remaining new+std::move -> o3tl::make_uniqueStephan Bergmann1-2/+1
Change-Id: I6a9a46d37bf09f6b497d591498c40103f4c96f59
2016-06-14starmath: SmTableNode has type NTABLETakeshi Abe1-3/+4
Change-Id: I406b38f41c694d7b4df4d2bc78e7731eb2ef02e4 Reviewed-on: https://gerrit.libreoffice.org/26210 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-13starmath: SmElement owns SmNodeTakeshi Abe3-12/+15
SmNodePointer is no longer used, so drop it. Change-Id: I5194cc7e1d9e551f4131cbcac0d84351bb2f2eab Reviewed-on: https://gerrit.libreoffice.org/26146 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-10Clean up uses of Any::getValue() in starmathStephan Bergmann1-14/+13
Change-Id: I2f3be0d3473331b8dbd5d008b79acfbfed65f847
2016-06-10Pass it by const refTakeshi Abe2-10/+16
Change-Id: I0c2b48a503e2501210af690f02858dade994a733 Reviewed-on: https://gerrit.libreoffice.org/26048 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-09tdf#94698 cleanup Makefiles. Get rid of udkapi and offapiGleb Mishchenko2-8/+2
This patch changes occurences in makefiles from $(eval $(call gb_CppunitTest_use_api,comphelper_test_config, \ udkapi \ offapi \ )) to $(eval $(call gb_CppunitTest_use_sdk_api,comphelper_test_config)) (corrected instead of abandon, jani) Change-Id: Ic96ec65d82d7452e288f05a8b6d576ef543b068e Reviewed-on: https://gerrit.libreoffice.org/23426 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-06-08remove "object is disposed" warning in OBroadcastHelperVar::removeListenerNoel Grandin1-0/+1
doesn't add anything of value. Change-Id: Ie2dea7c43570640284771c992d0072ab61de425e Reviewed-on: https://gerrit.libreoffice.org/25871 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-08starmath: The formula tree must be of type SmTableNodeTakeshi Abe6-25/+14
This also omits the last push & pop on m_aNodeStack at the end of parsing. Change-Id: Iefff8fa801ea7cb9015d6cba0d5a972dee675cb7 Reviewed-on: https://gerrit.libreoffice.org/26003 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-07starmath: Make ctors explicitTakeshi Abe9-38/+38
Change-Id: Ic1b7ce74f73bd41824e0af78675b4e18af32b1f5 Reviewed-on: https://gerrit.libreoffice.org/25960 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-06-06tools: rename SvStream::Read/Write to ReadBytes/WriteBytesMichael Stahl3-4/+4
Change-Id: Ib788a30d413436aa03f813aa2fddcbc4d6cd2f9a Reviewed-on: https://gerrit.libreoffice.org/25972 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-06-06Revert "Revert "Simplify sfx2 removing SfxModuleArr_Impl and dummy SfxModule ↵Noel Grandin1-1/+1
flag"" This reverts commit abde31a2bc68302e1afafe1fcc3f5d85369010f8. Since my analysis was completely incorrect, and the bug was elsewhere.
2016-06-06Revert "Simplify sfx2 removing SfxModuleArr_Impl and dummy SfxModule flag"Noel Grandin1-1/+1
This reverts commit e319ef1171dab61fff2201f5c1470ca09894c395. Apparently, at some point, someone added a 'bool bDummy' param to SfxModule, but only updated 2 of the 5 callsites. Since we're passing in pointers here, at the other call sites, the bDummy param evaluated to 'true'.
2016-06-06Simplify sfx2 removing SfxModuleArr_Impl and dummy SfxModule flagMark Page1-1/+1
The SfxModuleArr_Impl is a static container of SfxModule, since this is private to the SfxModule class it does not require wrapping. The bDummy flag is unused, and complicates the class Change-Id: Ib03b215543a0c37d4edf20b2d0fc141a783e115e Reviewed-on: https://gerrit.libreoffice.org/25859 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-06remove unused constants in HRC filesNoel Grandin1-33/+0
Change-Id: Ia034fb16ce75caeb69fd746d4a602cd5fcfef6d1 Reviewed-on: https://gerrit.libreoffice.org/25817 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-03Convert ComplexTextLayoutMode to scoped enumNoel Grandin2-5/+5
Change-Id: I257f8e15fec92f0701235a6fe4b6a2272498c4f1 Reviewed-on: https://gerrit.libreoffice.org/25667 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-03Null pointer dereferences must not happenTakeshi Abe1-1/+4
This is a follow-up of 09981cd6383ecb99e4b6c83b98b03af5cf3ff59b. Inspired by coverity#1362478 etc. Change-Id: If5507c52552393cccbb255cbf4b7e9f4b80db917 Reviewed-on: https://gerrit.libreoffice.org/25840 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-06-02coverity#1362478 Explicit null dereferencedCaolán McNamara1-36/+39
and coverity#1362479, coverity#1362480, coverity#1362481, coverity#1362482, coverity#1362483, coverity#1362485 Change-Id: Ia3a32b69bcbe5ac3e7cc50dacfa02e8bf1aab787
2016-06-02STREAM_STD_READ already includes StreamMode::NOCREATETakeshi Abe2-2/+2
Change-Id: Ib8c81056619a383cedc828b945c1802ff1ce42ca Reviewed-on: https://gerrit.libreoffice.org/25802 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-06-02Passing its const ref is sufficientTakeshi Abe1-2/+2
Change-Id: I85db4fcc3698d02ac6a7660042152092e6eeb99e Reviewed-on: https://gerrit.libreoffice.org/25752 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Page <aptitude@btconnect.com> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-31Use std::vector for SmCaretPosGraphTakeshi Abe3-107/+56
instead of employing ad hoc linked list and its iterator. Change-Id: Ibc4709a2e67aa805cf54117303c47d9a8a5eede9 Reviewed-on: https://gerrit.libreoffice.org/25699 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-05-31Prefix SmDocShell's membersTakeshi Abe3-136/+136
as well as remaning aInterpreter to maParser. Change-Id: I5f86737ffe05be981477e9dc65d433de0e1d5378 Reviewed-on: https://gerrit.libreoffice.org/25650 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-30starmath: Drop redundant IsFormulaArranged() checkTakeshi Abe3-15/+10
as ArrangeFormula() itself checks bIsFormulaArranged, and does nothing if it's on. Change-Id: Icc76208f6881d4de99e88feae5f19d7e8890e4e9 Reviewed-on: https://gerrit.libreoffice.org/25570 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-28starmath: Make some functions of SmDocShell publicTakeshi Abe2-11/+10
so that SmModel is no longer required to be a friend of SmDocShell. Change-Id: I7153a9ef13ae829710acf580ff0a9c30705aeb25 Reviewed-on: https://gerrit.libreoffice.org/25528 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-27starmath: Prefix members of SmSetSelectionVisitorTakeshi Abe2-65/+65
Change-Id: I3a5d195d65fe3a9b7b7859e3ad4c1d051628f44d Reviewed-on: https://gerrit.libreoffice.org/25491 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-26starmath: nodes in the clipboard are owened by SmClipboardTakeshi Abe2-51/+27
This also omits the bIsOnlyIfSelected parameter for former CloneLineToList(), which was true at its sole call site. Change-Id: Idb71323f68f13ecc90d430ec8e18e0eef766ae4b Reviewed-on: https://gerrit.libreoffice.org/25444 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-25(nearly) nothing uses GetUniqueId anymore, so remove it.Caolán McNamara1-1/+0
The odd one out is the usage in Formula, which attempts to restore focus to a particular window identified by an unique id. In this case restore focus by keeping a VclPtr to the desired window. Change-Id: I1dc335325c109d75745c6bba2e12662e6ae50638
2016-05-24starmath: Fix missing call of AnnotationSelection() for SmCursor::CopyTakeshi Abe3-0/+175
... with unit tests of Copy/Cut/Paste. Change-Id: I74dd6f235b52ef2c1388ea0d15d32af0fb30b2c8 Reviewed-on: https://gerrit.libreoffice.org/25362 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-23tdf#99984 render inferred mrow as group {}Regina Henschel2-5/+16
In case an element has no content, but has got an SmXMLRowContext, an empty group {} is inserted. The result is, that empty table cells are rendered without error, so that files, which use a mtable element to arrange things, no longer break on import. Change-Id: Iae158226e6478f2f9f397b8485044860ee8767c2 Reviewed-on: https://gerrit.libreoffice.org/25273 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-05-23starmath: This should be an assert()Takeshi Abe1-5/+1
Change-Id: Ia8dd75512168fd0775a8019fc514627a7353b056 Reviewed-on: https://gerrit.libreoffice.org/25231 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-21starmath: These functions are used in local onlyTakeshi Abe2-5/+6
Change-Id: I603be0a0890844019700f402ae79203ddc86c952 Reviewed-on: https://gerrit.libreoffice.org/25186 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-19starmath: Assign the proper token to SmTableNodeTakeshi Abe2-7/+6
... for a similar motivation with b43b79648f4fbcd815092268d1b7a5af9e278f52. Change-Id: I9fc42fc8e34958295eeec5759c072c8259fc38b6 Reviewed-on: https://gerrit.libreoffice.org/25122 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-05-19loplugin:unusedmethods in slideshow to svtoolsNoel Grandin2-14/+0
Change-Id: Icf0056e13c88d7d347e668adaeddd4ed72af85cf Reviewed-on: https://gerrit.libreoffice.org/25141 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-18starmath: Remove obsolete TG::DiscardedTakeshi Abe2-5/+4
which was originally TGDISCARDED until cc1a0ba927ad6f85103059aa8e6108017f436304, and its usage was gone with 9adcd261054a0e99ae032be5e7428e28a6663975. Change-Id: Idb01f30b7b140b17deead7c5a83128fa1f3bdf4c Reviewed-on: https://gerrit.libreoffice.org/25074 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: David Tardon <dtardon@redhat.com>
2016-05-18Convert ATTR_* to scoped enumTakeshi Abe2-27/+35
Change-Id: Icbf19ca281c260ba0795c7a8e5745d1e603ac8a5 Reviewed-on: https://gerrit.libreoffice.org/25050 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-17convert TG constants to scoped enumNoel Grandin6-373/+379
Change-Id: Ide719e8bb0d5aacc8d5da18f385b6373c7e48184 Reviewed-on: https://gerrit.libreoffice.org/24973 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-16tdf#98238, tdf#99591 reverted old commitJan Iversen1-8/+20
Revert "tdf#53472 Integral Symbols too small for some integrands" This reverts commit 746633bd0315939fd5e0b50c90692d356d2ed678. Revert on request from regina, since this commit caused 2 new bugs. The code of the reverted patch is kept in the file for further investigation, once it has been confirmed that it works. The file had 3 commits later, these are not affected by this commit Change-Id: I5b24a9b88ac909c83115e8c8a430dc768c9effd4 Reviewed-on: https://gerrit.libreoffice.org/24898 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Regina Henschel <rb.henschel@t-online.de> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-05-16Avoid C-style castTakeshi Abe1-1/+2
Change-Id: I777920cb2e9dcc3f2b20f5a96c88bf48fcb94895 Reviewed-on: https://gerrit.libreoffice.org/25024 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-14Fix bogus nGroup for SmMatrixNode's tokenTakeshi Abe1-1/+0
It was an obvious mistake because TRGROUP's type is SmTokenType. Change-Id: Ib3b932482c61bd9568aa579d94f742d0dd0b5462 Reviewed-on: https://gerrit.libreoffice.org/24948 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-13Resolves: tdf#99556 if the num of arguments is not 1 infer a rawCaolán McNamara3-5/+18
not just if the num of arguments is > 1 Change-Id: If0cae16cc52685315708ac3b2b8456ede7c1a6ce
2016-05-13convert SHOWTRACK to scoped enumNoel Grandin1-1/+1
Change-Id: Ibf06f6c79f80d7265ae3239c1ea0088118e92e21 Reviewed-on: https://gerrit.libreoffice.org/24894 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-13starmath: SmMatrixNode should have the node TMATRIXTakeshi Abe2-3/+5
... rather than the one after its closing brace. So far having an arbitrary node is OK for SmMatrixNode because existing code does not refer to it, but it is saner to keep relevant nodes in the syntax tree. Change-Id: Ia5412454c996eedde1545cd8a2f323eb95593eb9 Reviewed-on: https://gerrit.libreoffice.org/24909 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-12Convert FLG_* to scoped enumTakeshi Abe2-29/+39
Change-Id: If767fff10a0bb28735578b4ec3dcf7b9e2d326a2 Reviewed-on: https://gerrit.libreoffice.org/24872 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
2016-05-11clang-tidy modernize-loop-convert in scripting to svtoolsNoel Grandin9-38/+35
Change-Id: I98229d14109cf243839d632feabde1391ea9bad5 Reviewed-on: https://gerrit.libreoffice.org/24847 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-11starmath: Drop unused non-const variant of GetLeftMostTakeshi Abe2-17/+10
... resulting in one less const_cast. Change-Id: I3dad339f25524fc22e7a02b08ee7ec01b341e991 Reviewed-on: https://gerrit.libreoffice.org/24810 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
generated by cgit v1.2.3 (git 2.39.1) at 2024-04-28 11:45:23 +0000