[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: koffice/plugins/textshape
From:       Inge Wallin <inge () lysator ! liu ! se>
Date:       2009-07-19 12:09:49
Message-ID: 200907191409.49366.inge () lysator ! liu ! se
[Download RAW message or body]

On Saturday 18 July 2009 23:36:17 Elvis Stansvik wrote:
> SVN commit 998991 by astan:
>
> Doing some detailed commenting of this code, for my own and
> Casper's sanity ;)
>
>  M  +54 -10    Layout.cpp
>
>
> --- trunk/koffice/plugins/textshape/Layout.cpp #998990:998991
> @@ -329,43 +329,87 @@
>      else
>          m_isRtl =  dir == KoText::RightLeftTopBottom || dir ==
> KoText::PerhapsRightLeftTopBottom;
>
> -    // tables.
> +    // Check if we are inside a table.
>      QTextCursor tableFinder(m_block);
>      QTextTable *table = tableFinder.currentTable();
>      if (table) {
> +        /*
> +         * The current table is different from the one that is currently
> +         * set on the table layout, which means we have entered a table.
> +         */

Hmm, I must admit that I'm not 100% sure, but I seem to remember that our 
style guide says that even multi line comments should use // instead of /* */.

	-Inge

[prev in list] [next in list] [prev in thread] [next in thread]