[Libreoffice-commits] .: svtools/inc

Tor Lillqvist tml at kemper.freedesktop.org
Mon Aug 22 22:22:23 PDT 2011


 svtools/inc/svtools/table/tablecontrolinterface.hxx |   10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

New commits:
commit 172e6f6362827480754c3d54f336e30d10bac151
Author: Tor Lillqvist <tlillqvist at novell.com>
Date:   Tue Aug 23 08:18:57 2011 +0300

    Add comment with discussion on what existing comments claim

diff --git a/svtools/inc/svtools/table/tablecontrolinterface.hxx b/svtools/inc/svtools/table/tablecontrolinterface.hxx
index 74e9688..a187136 100644
--- a/svtools/inc/svtools/table/tablecontrolinterface.hxx
+++ b/svtools/inc/svtools/table/tablecontrolinterface.hxx
@@ -124,12 +124,20 @@ namespace svt { namespace table
     struct ColumnMetrics
     {
         /** the start of the column, in pixels. Might be negative, in case the column is scrolled out of the visible
-            area.
+            area. Note: see below.
         */
         long    nStartPixel;
 
         /** the end of the column, in pixels, plus 1. Effectively, this is the accumulated width of a all columns
             up to the current one.
+
+            Huh? Earlier you said that the nStartPixel of columns
+            scrolled out (to the left) of the visible area is
+            negative. Also, where is the promise that there is no gap
+            between columns? The above claim would be true only if the
+            first column always started at zero, and there is never a
+            gap. So these doc comments are inconsistent. How
+            surprising.
         */
         long    nEndPixel;
 


More information about the Libreoffice-commits mailing list