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

List:       calligra-devel
Subject:    Re: KoTableColumnAndRowStyleManager Question.
From:       Ganesh Paramasivam <ganesh () crystalfab ! com>
Date:       2011-02-28 9:20:53
Message-ID: AANLkTimy3-LUNf60CMFVfOE3m0P-tW=bVmy=wg9445Nf () mail ! gmail ! com
[Download RAW message or body]

Yes, you are right. I missed looking at this declaration
QExplicitlySharedDataPointer<Private> d;

Thanks a lot..

- Ganesh

On Mon, Feb 28, 2011 at 2:30 PM, Elvis Stansvik <elvstone@gmail.com> wrote:
> 2011/2/28 Ganesh Paramasivam <ganesh@crystalfab.com>:
>> Casper,
>>
>> A Quick question
>>
>> In KoTableColumnAndRowStyleManager.cpp we have a getManager() function
>> with the prototype
>>
>> KoTableColumnAndRowStyleManager
>> KoTableColumnAndRowStyleManager::getManager(QTextTable *table)
>>
>> Shouldn't the function return a reference or a pointer ? like this
>>
>> KoTableColumnAndRowStyleManager&
>> KoTableColumnAndRowStyleManager::getManager(QTextTable *table)
>>
>> If I were to set a row-style or a column-style for a table and do this
>> KoTableColumnAndRowStyleManager::getManager(table)->setRowStyle(0, rowStyle)
>> I would be setting these values on a copy of the return value ( and
>> not on the actual Manager ).
>
> Yes you would.
>
> Not sure, but I guess it works because KoTableColumnAndRowStyleManager
> is explicitly shared. I.e. all KoTableColumnAndRowStyleManager
> instances share the same data. Right Casper?
>
> Elvis
>
>>
>> Am I missing something ?
>>
>> Thanks,
>> Ganesh
>> _______________________________________________
>> calligra-devel mailing list
>> calligra-devel@kde.org
>> https://mail.kde.org/mailman/listinfo/calligra-devel
>>
> _______________________________________________
> calligra-devel mailing list
> calligra-devel@kde.org
> https://mail.kde.org/mailman/listinfo/calligra-devel
>
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic