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