diff options
author | Andras Timar <andras.timar@collabora.com> | 2019-02-21 12:30:48 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2019-02-21 14:28:43 +0100 |
commit | 1f0206d940cd8f7fb627a59cfe4165c0bfebaf46 (patch) | |
tree | 9e19b63ab2c4014a049c5f0f4a0fb215862205de /oox/source/drawingml/table | |
parent | a1deb4f6f159c1e9cfbde8407399746c1a2ce556 (diff) |
Fix code formatting, typos and grammar in comments, follow-up of 48ef20f2039
Change-Id: Ia5fa581936e33be1d163dca349ddbe006a79d5db
Reviewed-on: https://gerrit.libreoffice.org/68145
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'oox/source/drawingml/table')
-rw-r--r-- | oox/source/drawingml/table/tableproperties.cxx | 65 |
1 files changed, 33 insertions, 32 deletions
diff --git a/oox/source/drawingml/table/tableproperties.cxx b/oox/source/drawingml/table/tableproperties.cxx index 5bd61e8d97ae..b7b25a0e7886 100644 --- a/oox/source/drawingml/table/tableproperties.cxx +++ b/oox/source/drawingml/table/tableproperties.cxx @@ -270,20 +270,21 @@ const TableStyle& TableProperties::getUsedTableStyle( const ::oox::core::XmlFilt return *pTableStyle; } -void TableProperties::pushToPropSet( const ::oox::core::XmlFilterBase& rFilterBase, - const Reference < XPropertySet >& xPropSet, const TextListStylePtr& pMasterTextListStyle ) +void TableProperties::pushToPropSet(const ::oox::core::XmlFilterBase& rFilterBase, + const Reference<XPropertySet>& xPropSet, + const TextListStylePtr& pMasterTextListStyle) { - uno::Reference< XColumnRowRange > xColumnRowRange( - xPropSet->getPropertyValue("Model"), uno::UNO_QUERY_THROW ); + uno::Reference<XColumnRowRange> xColumnRowRange(xPropSet->getPropertyValue("Model"), + uno::UNO_QUERY_THROW); - CreateTableColumns( xColumnRowRange->getColumns(), mvTableGrid ); - CreateTableRows( xColumnRowRange->getRows(), mvTableRows ); + CreateTableColumns(xColumnRowRange->getColumns(), mvTableGrid); + CreateTableRows(xColumnRowRange->getRows(), mvTableRows); std::unique_ptr<TableStyle> xTableStyleToDelete; - const TableStyle& rTableStyle( getUsedTableStyle( rFilterBase, xTableStyleToDelete ) ); + const TableStyle& rTableStyle(getUsedTableStyle(rFilterBase, xTableStyleToDelete)); sal_Int32 nRow = 0; - for (auto & tableRow : mvTableRows) + for (auto& tableRow : mvTableRows) { sal_Int32 nColumn = 0; sal_Int32 nColumnSize = tableRow.getTableCells().size(); @@ -293,50 +294,50 @@ void TableProperties::pushToPropSet( const ::oox::core::XmlFilterBase& rFilterBa { TableCell& rTableCell(tableRow.getTableCells().at(nColIndex)); - if ( !rTableCell.getvMerge() && !rTableCell.gethMerge() ) + if (!rTableCell.getvMerge() && !rTableCell.gethMerge()) { - uno::Reference< XTable > xTable( xColumnRowRange, uno::UNO_QUERY_THROW ); + uno::Reference<XTable> xTable(xColumnRowRange, uno::UNO_QUERY_THROW); bool bMerged = false; - if ( ( rTableCell.getRowSpan() > 1 ) || ( rTableCell.getGridSpan() > 1 ) ) + if ((rTableCell.getRowSpan() > 1) || (rTableCell.getGridSpan() > 1)) { - MergeCells( xTable, nColumn, nRow, rTableCell.getGridSpan(), rTableCell.getRowSpan() ); + MergeCells(xTable, nColumn, nRow, rTableCell.getGridSpan(), + rTableCell.getRowSpan()); - if(rTableCell.getGridSpan() > 1) + if (rTableCell.getGridSpan() > 1) { nRemovedColumn = (rTableCell.getGridSpan() - 1); - // MergeCells removes columns. So our loop don't know that removed column and we are skip the handlingthat removed column(s). + // MergeCells removes columns. Our loop does not know about those + // removed columns and we skip handling those removed columns. nColIndex += nRemovedColumn; - bMerged = true; // it will adjust new column number after push current column's props with pushToXCell. + // It will adjust new column number after push current column's + // props with pushToXCell. + bMerged = true; } } - Reference< XCellRange > xCellRange( xTable, UNO_QUERY_THROW ); - Reference < XCell > xCell; + Reference<XCellRange> xCellRange(xTable, UNO_QUERY_THROW); + Reference<XCell> xCell; - if(nRemovedColumn) + if (nRemovedColumn) { try { - xCell = xCellRange->getCellByPosition( nColumn, nRow ); + xCell = xCellRange->getCellByPosition(nColumn, nRow); } - catch(Exception&) //Exception can come from TableModel::getCellByPosition when an column removed while merging columns. So adjust again here. + // Exception can come from TableModel::getCellByPosition when a column + // is removed while merging columns. So adjust again here. + catch (Exception&) { - xCell = xCellRange->getCellByPosition( nColumn-nRemovedColumn, nRow ); + xCell = xCellRange->getCellByPosition(nColumn - nRemovedColumn, nRow); } } else - xCell = xCellRange->getCellByPosition( nColumn, nRow ); - - rTableCell.pushToXCell( rFilterBase, - pMasterTextListStyle, - xCell, - *this, - rTableStyle, - nColumn, - tableRow.getTableCells().size()-1, - nRow, - mvTableRows.size()-1 ); + xCell = xCellRange->getCellByPosition(nColumn, nRow); + + rTableCell.pushToXCell(rFilterBase, pMasterTextListStyle, xCell, *this, rTableStyle, + nColumn, tableRow.getTableCells().size() - 1, nRow, + mvTableRows.size() - 1); if (bMerged) nColumn += nRemovedColumn; } |