Review Request 109189: [Kig] Merge nowarnings Branch into master

Commit Hook null at kde.org
Sun Mar 3 08:30:50 UTC 2013


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


This review has been submitted with commit 55b18a3f9ce371cc6cb7ce3d2eeeeb4c83a80f1d by David E. Narváez to branch master.

- Commit Hook


On March 1, 2013, 2:49 a.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109189/
> -----------------------------------------------------------
> 
> (Updated March 1, 2013, 2:49 a.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Description
> -------
> 
> I started the nowarnings branch with the goal of eliminating all compilation branches in Kig. This will hopefully improve the perceived quality of the current code and help developers spot warnings when submitting new code.
> 
> This branch currently compiles without warnings in GCC 4.{6,7,8} and Clang 3.2. While most changes deal with the -Woverloaded-virtual warning in GCC, some changes are worth double  checking: kig.cpp, main.cpp and calcpaths.cc.
> 
> 
> Diffs
> -----
> 
>   filters/asyexporter.cc cbb872f1c3873019d6ca686cf75c7d1613be6a7c 
>   filters/asyexporterimpvisitor.h 630d0915d5211f525f37a8f1f546509ec4c61aef 
>   filters/cabri-filter.cc 411b450355981447f3e901551897968899e03396 
>   filters/latexexporter.cc 9edd34cc88eaf14b479007c4e2e8a1994e9656db 
>   filters/latexexporteroptions.cc be127827215f8539eebd615c94d09eef643638bc 
>   filters/pgfexporterimpvisitor.h 165f2df8ad9866fd1e43ae4160b90abcf2e74281 
>   filters/xfigexporter.cc 4c26cbff3a8f27b7a07570cb146c59895942c3f3 
>   kig/kig.cpp af1b9fe775f0115947cb427d3e6d115f88d7507c 
>   kig/main.cpp 94277a308cd83737ba63f803c6852204b05dd0b5 
>   misc/calcpaths.cc 4facd8fb797607fc1e8dc0f13752c98c92a78e9b 
>   misc/special_constructors.cc 23325bdca937b6947d73ccd80ae957c8f3f24346 
>   modes/base_mode.h e1365c6ebea58e3e010f19ba0a1118eb79970a9b 
>   modes/construct_mode.h ba069069a7ce62859a2eed43a8d9c4a9e3e78807 
>   modes/macro.h 2d58141c1a3c04877c41343fe24d7b3d10a17618 
>   modes/normal.h 1e11e6f67227eb4d9232470d080877bd9e1ec0e3 
>   objects/conic_types.h 849871f19d13c9b30209d72b3bffc9409cc3d769 
>   objects/line_type.h 5a95552edeb0e73aa13138d7815ed871a65d1347 
>   scripting/python_scripter.cc 7b882bda1b071533407cd880b93586fb2fb2cb92 
>   scripting/script_mode.h 4cbfd73763bef44061fe5a09c6254d95fa884a95 
> 
> Diff: http://git.reviewboard.kde.org/r/109189/diff/
> 
> 
> Testing
> -------
> 
> 1. Compiled and ran the application using GCC 4.6 (32 bits), GCC 4.{7,8} and Clang in (64 bits).
> 2. Compiled and ran with the --convert-to-native option
> 
> 
> Thanks,
> 
> David Narváez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20130303/7df9fa50/attachment.html>


More information about the kde-edu mailing list