[Libreoffice-commits] .: sw/source

Julien Nabet serval2412 at kemper.freedesktop.org
Tue Jul 10 14:32:33 PDT 2012


 sw/source/core/docnode/ndtbl.cxx |    5 +++++
 1 file changed, 5 insertions(+)

New commits:
commit dd0e782d1176f6f52d121fdc920f015ea7647d45
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Tue Jul 10 23:31:37 2012 +0200

    Add some explanation (thank you David!)
    
    (see http://nabble.documentfoundation.org/Some-strange-lines-by-taking-a-look-at-the-bt-of-fdo-51916-tp3994561p3994639.html)
    
    Change-Id: I93830306c70155532f461957991df8dde29540da

diff --git a/sw/source/core/docnode/ndtbl.cxx b/sw/source/core/docnode/ndtbl.cxx
index 118eeb2..ab96959 100644
--- a/sw/source/core/docnode/ndtbl.cxx
+++ b/sw/source/core/docnode/ndtbl.cxx
@@ -1357,7 +1357,12 @@ SwTableNode* SwNodes::TextToTable( const SwNodes::TableRanges_t & rTableNodes,
                ++aCellEndIdx;
                SwStartNode* pSttNd = new SwStartNode( aTmpIdx, ND_STARTNODE,
                                             SwTableBoxStartNode );
+
+                // Quotation of http://nabble.documentfoundation.org/Some-strange-lines-by-taking-a-look-at-the-bt-of-fdo-51916-tp3994561p3994639.html
+                // SwNode's constructor adds itself to the same SwNodes array as the other node (pSttNd).
+                // So this statement is only executed for the side-effect.
                 new SwEndNode( aCellEndIdx, *pSttNd );
+
                 //set the start node on all node of the current cell
                 SwNodeIndex aCellNodeIdx = aCellIter->aStart;
                 for(;aCellNodeIdx <= aCellIter->aEnd; ++aCellNodeIdx )


More information about the Libreoffice-commits mailing list