2009/7/19 Inge Wallin <inge@lysator.liu.se>:
> 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 /* */.

Ah, I'll make sure to fix those and some others up then.

Elvis

>
>        -Inge
>
>
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel