Review Request 118984: KoPart argument of KoDocument and descendants should not be optional
Inge Wallin
inge at lysator.liu.se
Sun Jun 29 10:51:03 BST 2014
> On June 29, 2014, 4:45 a.m., Inge Wallin wrote:
> > In general looks good. But shouldn't the same idea be used in the other apps too?
>
> Boudewijn Rempt wrote:
> It's something that is going away in the mvc branch in any case.
Ok. I would love to see the mvc branch merged as soon as possible.
- Inge
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118984/#review61158
-----------------------------------------------------------
On June 27, 2014, 10:25 p.m., Jarosław Staniek wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118984/
> -----------------------------------------------------------
>
> (Updated June 27, 2014, 10:25 p.m.)
>
>
> Review request for Calligra and Boudewijn Rempt.
>
>
> Repository: calligra
>
>
> Description
> -------
>
> KoPart argument of KoDocument and descendants should not be optional
>
> Currently KoPart is required, there is even assertion for, so express this in the API.
> Current API apparently caused one crash (in spreadsheet plugin of Kexi).
>
> Also put 2 more assertions as early as possible.
>
> Finally, fixed outdated docs.
>
>
> Diffs
> -----
>
> sheets/DocBase.h 946e9cd9bdbf414430bd7914bc6bb5bf3e57c3df
> sheets/DocBase.cpp b01a2004c15d2a03201e527af3ac3255f8c75646
> sheets/part/Doc.h bf8341cafd0f6f79668870043af1169ce76df0ca
> sheets/part/Doc.cpp 44fd7ab5e237676296a28590e938a01669f0bb86
>
> Diff: https://git.reviewboard.kde.org/r/118984/diff/
>
>
> Testing
> -------
>
> Builds, nobody is used the argument-less constructor (after https://git.reviewboard.kde.org/r/118983/ is pushed)
>
>
> Thanks,
>
> Jarosław Staniek
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140629/06c605d0/attachment.htm>
More information about the calligra-devel
mailing list