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

David Narváez david.narvaez at computer.org
Fri Mar 1 02:49:58 UTC 2013


-----------------------------------------------------------
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.


Changes
-------

Addressing all opened issues


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 (updated)
-----

  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/20130301/64fe3831/attachment.html>


More information about the kde-edu mailing list