Review Request: Patch for using KoAbstractionController directly from FreOffice

Mani N C maninc at gmail.com
Tue Feb 1 06:43:36 GMT 2011


My bad, I agree we won't be able to connect signals. The problem I have is
when f-office is compiled out-of-tree the gives errors. I am trying to fix
this in such a way that it would compile in-source and out-of-tree. Thanks
for your comments, I will try to emit pure virtual methods and see.

Br,
Mani

On Mon, Jan 31, 2011 at 7:23 PM, Jarosław Staniek <staniek at kde.org> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100462/
>
> One of the solutions is to have emit*() pure virtual methods in the controller and implement them in f-office to emit real signals.
>
> Sebastian, you wrote in ba54ba08 3 days ago that something did not build. But it did before...
>
>
> - Jarosław
>
> On January 31st, 2011, 12:07 p.m., Mani Chandrasekar wrote:
>   Review request for Calligra.
> By Mani Chandrasekar.
>
> *Updated Jan. 31, 2011, 12:07 p.m.*
> Description
>
> This patch removes KoAbstraction class which implements KoAbstractionContorller.
>
> Since we are reimplementing most of the functions in MainWindow.cpp I have removed KoAbstraction class and moved all the signals to MainWindow
> I feel the implementation should be in freoffice code instead of abstraction library.
>
> Is there any possible drawbacks in this approach?
>
>   Diffs
>
>    - tools/CMakeLists.txt (d4e6ab5)
>    - tools/f-office/CMakeLists.txt (a212bc0)
>    - tools/f-office/MainWindow.h (f7b6149)
>    - tools/f-office/MainWindow.cpp (549b7d1)
>
> View Diff <http://git.reviewboard.kde.org/r/100462/diff/>
>



-- 
Mani Chandrasekar
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110201/9f823d51/attachment.htm>


More information about the calligra-devel mailing list