/icon-themes/breeze/reportdesign/

lication/atom+xml'/>
summaryrefslogtreecommitdiff
path: root/sc/inc/colcontainer.hxx
AgeCommit message (Collapse)Author
2017-08-25dynamic column container: more efficient loops over all colsNoel Grandin
Create an ScColumnsRange class that returns a pair of (start,end) iterators to go through the list of currently allocated columns. This is a fairly thing wrapper around the underlying std::vector, so it should be fairly efficient (two pointers, and pointer increment for iteration). If this style of iteration is acceptable, I'll go through the rest of the code that does: for (SCCOL nCol=0; nCol<MAXCOLCOUNT; nCol++) type stuff, and change it to use ScColumnsRange. Change-Id: I81501c69b7f5566c6204dde0d87a6fe0deb9743c Reviewed-on: https://gerrit.libreoffice.org/41413 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-07-04loplugin:unusedfields in sc part3Noel Grandin
Change-Id: I7323e8644e5d336d52fa21b063264b0847b5e780 Reviewed-on: https://gerrit.libreoffice.org/39064 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-06-06add COVERITY_NOEXCEPT_FALSECaolán McNamara
to markup dtors that coverity warns might throw exceptions which won't throw in practice, or where std::terminate is an acceptable response if they do Change-Id: I32b94814e8245372e1d1dc36be0d81e3564042f4 Reviewed-on: https://gerrit.libreoffice.org/38318 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-05-03tdf#107255: detect whether the range has only one data cell.Kohei Yoshida
Change-Id: I030961d9d38b092ffdc966baa10decae0c2d070d Reviewed-on: https://gerrit.libreoffice.org/37178 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Kohei Yoshida <libreoffice@kohei.us>
2017-02-24tdf#50916 Allow dynamically increase number of columns according to needsBartosz Kosiorek
Change-Id: I08b1d70b6aafb01738bb5dec3f4eafd7b21e6bb5 Reviewed-on: https://gerrit.libreoffice.org/33724 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
2016-11-30tdf#50916 : Refactor table1.cxx wherever there is column accessDennis Francis
This is still a WIP for table1.cxx, but want to get some comments as I proceed. Added back() method to ScColContainer to avoid aCol[aCol.size()-1] usages. Looks like we will need to initialize column container with at least one column in the ScTable constructor because we need to use a lot of "aCol.size() - 1" expression. Change-Id: I5f0fa3d2daed67cb623641b273d2727561ab3d49 Reviewed-on: https://gerrit.libreoffice.org/31125 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-05-26loplugin:unusedmethodsNoel Grandin
Change-Id: Ib36bc0e87d00abb638cbfec511cd13b6179eabda Reviewed-on: https://gerrit.libreoffice.org/25431 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-24loplugin:unusedmethods in scNoel Grandin
Change-Id: Ic378126a30be853d10fe174c451cee3c6ded404f Reviewed-on: https://gerrit.libreoffice.org/25109 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-05Avoid ambiguities among operator[] overloadsStephan Bergmann
Change-Id: Ide7446e1aebe698277b13cc7eeb39efe16af3ec2
2016-02-04Patch#1 : Dynamic column container in the pursuit of tdf#50916Dennis Francis
In this patch dynamic column structure is introduced, basically wrapping std::vector<ScColumn*>. In ScTable the column container is pre-allocated with MAXCOL + 1 columns so that the rest of the code that uses need not change for now. So for now the new column container will still behave like the static one. The plan is to *incrementally* modify all instances of iterations over columns to make use of the dynamic column container and also to address the issues mentioned in the thread : http://nabble.documentfoundation.org/tdf-50916-Calc-Dynamic-column-container-td4162663.html The final step would be to remove the pre-allocation of the container in ScTable constructor and increase of MAXCOLCOUNT Change-Id: I3ff18cdebc99d8348e06a76f287d1d30279366bd Reviewed-on: https://gerrit.libreoffice.org/21620 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Jenkins <ci@libreoffice.org>