[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