Review Request 109260: Normalize SIGNAL and SLOT signatures in Stage

Thorsten Zachmann t.zachmann at zagge.de
Sun Mar 3 18:14:22 GMT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109260/#review28486
-----------------------------------------------------------


What is the difference in all the files that show no difference in reviewboard?


stage/part/KPrViewModeSlidesSorter.cpp
<http://git.reviewboard.kde.org/r/109260/#comment21279>

    Is this removing of whitespace necessary? Having a blank after a , makes it more readable?


- Thorsten Zachmann


On March 3, 2013, 3:30 p.m., Inge Wallin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109260/
> -----------------------------------------------------------
> 
> (Updated March 3, 2013, 3:30 p.m.)
> 
> 
> Review request for Calligra and Thorsten Zachmann.
> 
> 
> Description
> -------
> 
> This is a normalization of SIGNAL and SLOT calls that is done with fixsrc.py.  By now I think it's 99% certain that it's correct but I still want somebody who knows the code to look through it.
> 
> 
> Diffs
> -----
> 
>   stage/part/KPrAnimationDirector.cpp 4b3a50d 
>   stage/part/KPrHtmlExport.cpp 9a9fc1c 
>   stage/part/KPrPart.cpp 84e4867 
>   stage/part/KPrPresentationDrawWidget.cpp 28c0264 
>   stage/part/KPrPresentationTool.cpp 3875f82 
>   stage/part/KPrPresenterViewSlidesInterface.cpp 1980981 
>   stage/part/KPrPresenterViewToolWidget.cpp a4243c7 
>   stage/part/KPrPresenterViewWidget.cpp d699c66 
>   stage/part/KPrView.cpp 078b34b 
>   stage/part/KPrViewAdaptor.cpp 1fd9051 
>   stage/part/KPrViewModePreviewPageEffect.cpp 45e33a6 
>   stage/part/KPrViewModeSlidesSorter.cpp 55b0e22 
>   stage/part/dockers/KPrPageLayoutDocker.cpp e0e3113 
>   stage/part/dockers/KPrPreviewWidget.cpp 34bca0e 
>   stage/part/tests/ModelTest.cpp 2494481 
>   stage/part/tools/animationtool/KPrAnimationSelectorWidget.cpp e603461 
>   stage/part/tools/animationtool/KPrAnimationsTimeLineView.cpp 98add38 
>   stage/part/tools/animationtool/KPrClickActionDocker.cpp 39167fd 
>   stage/part/tools/animationtool/KPrPageEffectDocker.cpp e892dd1 
>   stage/part/tools/animationtool/KPrShapeAnimationDocker.cpp e6039ae 
>   stage/part/ui/KPrConfigureSlideShowDialog.cpp 1b046bd 
>   stage/plugins/eventactions/sound/KPrSoundEventAction.cpp 1fbb3f4 
>   stage/plugins/eventactions/sound/KPrSoundEventActionWidget.cpp 34e4267 
> 
> Diff: http://git.reviewboard.kde.org/r/109260/diff/
> 
> 
> Testing
> -------
> 
> It builds just like before.  No new failing tests.
> 
> 
> Thanks,
> 
> Inge Wallin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130303/370a2fc1/attachment.htm>


More information about the calligra-devel mailing list