Home >  Krazy Code Checker >  bundled-apps >  koffice >  kplato

krazy2 Analysis

Checkers Run = 45
Files Processed = 373
Total Issues = 218 ...as of March 14 2010 03:53:27 CET

• For File Type desktop
  1. Check that file ends with a newline [endswithnewline]...okay!
  2. Check for invalid icon names [iconnames]...okay!
  3. Check for spelling errors [spelling]...okay!
  4. Validates desktop files using 'desktop-file-validate' [validate]...okay!
• For File Type c++
  1. Check for TRUE and FALSE macros [captruefalse]...okay!
  2. Check for an acceptable copyright [copyright]...OOPS! 1 issues found!
     • libs/models/kcalendar/kdatetable.cpp: missing tags

     All source files must contain a copyright header which identifies the copyright holder(s) together with a e-mail address that can be used to reach the copyright holder. One copyright holder per line, with real email addresses please. For details regarding KDE's licensing policy please visit http://techbase.kde.org/Policies/Licensing_Policy. A typical copyright looks like: "Copyright 2002,2005-2006 Joe Hacker "

  3. Check for cpp macros and usage [cpp]...OOPS! 15 issues found!
     • workpackage/kplatowork_export.h: O/S or Compiler specific macro line# 27,32,64,46,32,64,66,32,64 (9)
     • kplato_export.h: O/S or Compiler specific macro line# 27,32,64,48,32,64 (6)

     C++ source files and non-installed headers should NOT use cpp conditionals that check for a certain O/S or compiler; instead use CMake HAVE_foo macros. We want to check for features discovered during CMake time rather than for a specific O/S.

  4. Check for code that should be considered crashy. [crashy]...OOPS! 13 issues found!
     • libs/ui/reports/reportview.cpp: line# 598 (1)
     • libs/ui/kptganttview.cpp: line# 854,977 (2)
     • libs/ui/kptaccountsview.cpp: line# 213 (1)
     • libs/ui/kpttaskeditor.cpp: line# 1119 (1)
     • libs/ui/kptdocumentspanel.cpp: line# 111 (1)
     • workpackage/part.cpp: line# 224 (1)
     • workpackage/view.cpp: line# 331,393,433,450 (4)
     • workpackage/taskworkpackageview.cpp: line# 317 (1)
     • kptview.cpp: line# 2551 (1)

     Looks for not using QPointers when showing modal dialogs via exec(), as discussed in http://www.kdedevelopers.org/node/3919

  5. Check single-char QString operations for efficiency [doublequote_chars]...OOPS! 27 issues found!
     • libs/kernel/kptresource.cpp: line# 890,913 (2)
     • libs/kernel/kpttask.cpp: line# 1043,1055,1099,1262,1274 (5)
     • libs/kernel/kptwbsdefinition.cpp: line# 38 (1)
     • libs/models/kptpertcpmmodel.cpp: line# 236,746 (2)
     • libs/models/kptresourceappointmentsmodel.cpp: line# 800,817,837,858 (4)
     • libs/models/kptnodeitemmodel.cpp: line# 2921,4168 (2)
     • libs/models/kpttaskstatusmodel.cpp: line# 548 (1)
     • libs/models/kptcalendarmodel.cpp: line# 385,837 (2)
     • libs/models/kptrelationmodel.cpp: line# 374 (1)
     • libs/models/kptdocumentmodel.cpp: line# 359 (1)
     • libs/models/kptaccountsmodel.cpp: line# 72,363 (2)
     • libs/models/kptresourcemodel.cpp: line# 941 (1)
     • libs/models/kptschedulemodel.cpp: line# 696 (1)
     • libs/models/kptresourceallocationmodel.cpp: line# 817 (1)
     • libs/ui/kptscheduleeditor.cpp: line# 461 (1)

     Adding single characters to a QString is faster if the characters are QChars and not QStrings. For example: QString path = oldpath + "/" + base is better written as QString path = oldpath + '/' + base. Same holds for arguments to QString::startsWith(), QString::endsWith(), QString::remove(), QString::section(), and QString::split(). Use QString::remove() instead of QString::replace(foo,"")

  6. Check for QString compares to "" [emptystrcompare]...okay!
  7. Check that file ends with a newline [endswithnewline]...okay!
  8. Check for C++ ctors that should be declared 'explicit' [explicit]...OOPS! 1 issues found!
     • libs/ui/kptlocaleconfigmoneydialog.h: line# 39 (1)

     Make all C++ class constructors that can be used with only one required argument 'explicit' to minimize wrong use of the class. Do this to avoid mistaken implicit constructor ambiguities. Copy constructors should not be explicit.

  9. Check for foreach loop issues [foreach]...OOPS! 37 issues found!
     • libs/kernel/kptaccount.cpp: non-const ref iterator line# 737 (1)
     • libs/kernel/kptduration.cpp: non-const ref iterator line# 333 (1)
     • libs/kernel/kptresource.cpp: values or keys iteration line# 343,1096 (2)
     • libs/kernel/kpttask.cpp: values or keys iteration line# 2716,2719,2827,2846,2862,3052,3083,3114,3176,3335,3348,3597 (12)
     • libs/kernel/kptnode.cpp: values or keys iteration line# 99 (1)
     • libs/kernel/kptproject.cpp: values or keys iteration line# 1533,1591 (2)
     • libs/kernel/kptcommand.cpp: values or keys iteration line# 3319 (1)
     • libs/models/kcalendar/kdatetable.cpp: non-const ref iterator line# 193 (1)
     • libs/models/kptnodechartmodel.cpp: values or keys iteration line# 221,250,276 (3)
     • libs/models/kptcalendarmodel.cpp: values or keys iteration line# 325,352 (2)
     • libs/models/kptaccountsmodel.cpp: values or keys iteration line# 505,1083 (2)
     • libs/ui/kpttaskprogresspanel.cpp: values or keys iteration line# 171 (1)
     • libs/ui/kpttaskeditor.cpp: non-const ref iterator line# 1021 (1)
     • libs/ui/kptusedefforteditor.cpp: values or keys iteration line# 273 (1)
     • libs/ui/kptmilestoneprogresspanel.cpp: values or keys iteration line# 68 (1)
     • kptpart.cpp: values or keys iteration line# 575,592,706 (3)
     • kptview.cpp: values or keys iteration line# 1391,1498 (2)

     When not using POD types (int, double, pointer, ...) you should use const & for your foreach variables. There are two reasons for this: 1) Prevents you from the mistake of writing foreach loops that modify the list, that is 'foreach(Foo f, list) f.a = f.b = f.c = 0;' compiles but does not modify the contents of list 2) Saves a copy constructor call for each of the list elements

  10. Check validity of i18n calls [i18ncheckarg]...OOPS! 5 issues found!
      • plugins/schedulers/rcps/KPlatoRCPSPlugin.cpp: wrong argument count, have 2 need 1 line# 68 (1)
      • plugins/schedulers/rcps/KPlatoRCPSScheduler.cpp: gaps in placeholder numbering, (2 3 4) line# 319 (1)
      • plugins/schedulers/rcps/KPlatoRCPSScheduler.cpp: wrong argument count, have 2 need 1 line# 214 (1)
      • plugins/schedulers/rcps/KPlatoRCPSScheduler.cpp: wrong argument count, have 0 need 1 line# 121 (1)
      • plugins/schedulers/rcps/KPlatoRCPSScheduler.cpp: gaps in placeholder numbering, (2 3) line# 314 (1)

      Make the translators' job easier and detect problems in the usage of the i18n() calls. When the fix is not clear, check the Techbase article at http://techbase.kde.org/Development/Tutorials/Localization/i18n_Krazy for more information.

  11. Check for invalid icon names [iconnames]...OOPS! 1 issues found!
      • kptview.cpp: line# 2526 (1)

      Browse available icons with 'kdialog --geticon actions'. Request new icons at http://techbase.kde.org/Projects/Oxygen/Missing_Icons.

  12. Check for proper include directives [includes]...OOPS! 76 issues found!
      • plugins/schedulers/rcps/KPlatoRCPSScheduler.h: missing or improper include guard in header
      • plugins/scripting/ResourceGroup.cpp: include own header first line# 23 (1)
      • plugins/scripting/Calendar.cpp: include own header first line# 23 (1)
      • plugins/scripting/Node.cpp: include own header first line# 23 (1)
      • plugins/scripting/Resource.cpp: include own header first line# 23 (1)
      • plugins/scripting/Schedule.cpp: include own header first line# 23 (1)
      • plugins/scripting/Account.cpp: include own header first line# 23 (1)
      • libs/kernel/kptglobal.h: missing or improper include guard in header
      • libs/kernel/kptdocuments.h: missing or improper include guard in header
      • libs/kernel/kptschedulerplugin_macros.h: missing or improper include guard in header
      • libs/models/kcalendar/kdatetable_p.h: missing or improper include guard in header
      • libs/models/kcalendar/kdatetable.h: missing or improper include guard in header
      • libs/models/kcalendar/kdatepicker.h: missing or improper include guard in header
      • libs/models/kptdocumentmodel.h: missing or improper include guard in header
      • libs/models/kptnodeitemmodel.cpp: do not include QtModules line# 49 (1)
      • libs/models/kpttaskcompletedelegate.h: missing or improper include guard in header
      • libs/models/kptdurationspinbox.h: missing or improper include guard in header
      • libs/models/kptcommonstrings.h: missing or improper include guard in header
      • libs/models/kptflatproxymodel.h: missing or improper include guard in header
      • libs/ui/plotting/kplotwidget.h: missing or improper include guard in header
      • libs/ui/plotting/kplotaxis.h: missing or improper include guard in header
      • libs/ui/kptlocaleconfigmoneydialog.h: missing or improper include guard in header
      • libs/ui/reports/reportview.cpp: include own _p header second line# 23 (1)
      • libs/ui/kptganttview.cpp: duplicate includes line# 28,34 (2)
      • libs/ui/kpttaskeditor.h: missing or improper include guard in header
      • libs/ui/kptusedefforteditor.h: missing or improper include guard in header
      • libs/ui/kpttaskdescriptiondialog.cpp: duplicate includes line# 25,27 (2)
      • libs/ui/kptdocumentspanel.h: missing or improper include guard in header
      • libs/ui/kptrelationeditor.h: missing or improper include guard in header
      • libs/ui/kptaccountsview.cpp: duplicate includes line# 37,40 (2)
      • libs/ui/kptdependencyeditor.h: missing or improper include guard in header
      • libs/ui/kptsplitterview.h: missing or improper include guard in header
      • libs/ui/kpttaskeditor.cpp: duplicate includes line# 42,46 (2)
      • libs/ui/kptresourceassignmentview.h: duplicate includes line# 27,30,32 (3)
      • libs/ui/kptwbsdefinitionpanel.cpp: duplicate includes line# 29,32 (2)
      • libs/ui/kpttaskresourcespanelbase.ui.h: missing or improper include guard in header
      • libs/ui/kpttaskdescriptiondialog.h: missing or improper include guard in header
      • libs/ui/kptsummarytaskdialog.h: missing or improper include guard in header
      • libs/ui/kptmainprojectdialog.cpp: include own header first line# 24 (1)
      • libs/ui/kptviewbase.h: missing or improper include guard in header
      • workpackage/workpackage.cpp: duplicate includes line# 43,44 (2)
      • workpackage/application.cpp: duplicate includes line# 36,37,49,50,56,57 (6)
      • workpackage/part.cpp: duplicate includes line# 37,39,56,58 (4)
      • workpackage/mainwindow.cpp: duplicate includes line# 41,78,44,70,55,71,60,73 (8)
      • kptinsertfiledlg.h: missing or improper include guard in header
      • kptpart.h: missing or improper include guard in header
      • kptviewlist.h: missing or improper include guard in header
      • kptpart.cpp: duplicate includes line# 39,41 (2)
      • kptview.h: missing or improper include guard in header
      • kptaboutdata.h: missing or improper include guard in header
      • kptfactory.h: missing or improper include guard in header
      • kptconfigbehaviorpanelbase.ui.h: missing or improper include guard in header

      Use <..> to include installed headers; to include Qt headers from installed headers; cpp file should include their own headers first (but below config.h); other rules apply, see http://techbase.kde.org/Policies/Library_Code_Policy#Getting_.23includes_right. Use include guards in headers with appropriatedly encoded macro names.

  13. Check for an acceptable license [license]...OOPS! 3 issues found!
      • libs/ui/reports/reportdesignview.cpp: LGPL (v2.1+) (wrong address)
      • libs/ui/reports/reportdesignview.h: LGPL (v2.1+) (wrong address)
      • libs/ui/reports/reportpage.h: LGPL (v2.1+) (wrong address)

      Each source file must contain a license or a reference to a license which states under which terms the software may be used, modified and redistributed. For details regarding KDE's licensing policy please visit http://techbase.kde.org/Policies/Licensing_Policy.

  14. Check for assignments to QString::null [nullstrassign]...OOPS! 3 issues found!
      • libs/kernel/kptwbsdefinition.h: line# 55 (1)
      • libs/kernel/kptnode.cpp: line# 48 (1)
      • workpackage/part.cpp: line# 223 (1)

      Do not assign QString::null or QString() to a QString. Instead use the .clear() method. For example, "str = QString::null" becomes "str.clear()". When returning an empty string from a method use "return QString()" When passing an empty string use "QString()".

  15. Check for compares to QString::null or QString() [nullstrcompare]...okay!
  16. Check for C++ operators that should be 'const' [operators]...okay!
  17. Check for inappropriate pass-by-value function args [passbyvalue]...okay!
  18. Check for postfix usage of ++ and -- [postfixop]...okay!
  19. Check for dangerous or inefficient QByteArray usage [qbytearray]...okay!
  20. Check for Qt classes that should not be used [qclasses]...OOPS! 13 issues found!
      • libs/models/kcalendar/kdatetable_p.h: QLineEdit[KLineEdit]
      • libs/models/kcalendar/kdatepicker.cpp: QComboBox[KComboBox],QLineEdit[KLineEdit],QComboBox[KComboBox]
      • libs/models/kptitemmodelbase.cpp: QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox]
      • libs/models/kpttreecombobox.cpp: QComboBox[KComboBox]
      • libs/ui/kptsplitterview.cpp: QTabWidget[KTabWidget],QTabWidget[KTabWidget],QTabWidget[KTabWidget],QTabWidget[KTabWidget],QTabWidget[KTabWidget]
      • libs/ui/kptworkpackagesendpanel.cpp: QLineEdit[KLineEdit]
      • libs/ui/locale/localemon.cpp: QComboBox[KComboBox]
      • libs/ui/kptsplitterview.h: QTabWidget[KTabWidget]
      • libs/ui/kpttaskcostpanel.cpp: QComboBox[KComboBox]
      • libs/ui/kptwbsdefinitionpanel.cpp: QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox]
      • libs/ui/kptscheduleeditor.cpp: QTabWidget[KTabWidget]
      • libs/ui/kpttaskcostpanel.h: QComboBox[KComboBox]
      • kptviewlist.h: QComboBox[KComboBox]

      Don't use Qt 4 classes that are deprecated. Additionally make sure to use the KDE version of some Qt GUI elements to provide a consistent look and feel for the KDE desktop. The KDE classes are not just adding functionalities to the Qt base class and are mostly not even based on the Qt class. http://techbase.kde.org/Policies/API_to_Avoid Please refer to the API documentation for details before porting to the K classes.

  21. Check for Qt methods that should be avoided [qmethods]...okay!
  22. Check for QMIN and QMAX macros [qminmax]...OOPS! 1 issues found!
      • libs/ui/kptchartwidget.cpp: using QMAX line# 176 (1)

      Obsolete macros QMIN(), Q_MIN(), QMAX() and Q_MAX() should be replaced by the qMin() and qMax() functions.

  23. Check for classes that should use the 'Q_OBJECT' macro [qobject]...okay!
  24. Check for spelling errors [spelling]...OOPS! 5 issues found!
      • libs/models/kptflatproxymodel.cpp: line# 364[[indeces]] (1)
      • libs/models/kptflatproxymodel.h: line# 39[[usefull]],41[[indeces]],42[[indeces]] (3)
      • kptpart.cpp: line# 63[[convinience]] (1)

      Spelling errors in comments and strings should be fixed as they may show up later in API documentation, handbooks, etc. Misspelled strings make the translator's job harder. Please use US English.

  25. Check for strings used improperly or should be i18n. [strings]...OOPS! 1 issues found!
      • kptview.cpp: QLatin1String issues line# 605 (1)

      Some QString methods (like startsWith() and endsWith()) are more efficient if they are passed a QLatin1String, avoiding an implicit conversion from const char *.

  26. Check for system calls to replace by KDE or Qt equivalents [syscalls]...okay!
  27. Check for typedefs that should be replaced by Qt typedefs [typedefs]...okay!
• For File Type kpartgui
  1. Check that file ends with a newline [endswithnewline]...okay!
  2. Check validity of i18n calls [i18ncheckarg]...okay!
  3. Check for spelling errors [spelling]...okay!
  4. Validates kpartgui files using 'xmllint' [validate]...okay!
• For File Type messages
  1. Check for appending to rc.cpp [rcappend]...OOPS! 1 issues found!
     • plugins/scripting/Messages.sh: line# 2 (1)

     Always append extracted messages to the rc.cpp with '>>' shell redirection.

• For File Type designer
  1. Check that file ends with a newline [endswithnewline]...okay!
  2. Check for files that need to be fixed by 'fixuifiles' [fixuifiles]...okay!
  3. Check validity of i18n calls [i18ncheckarg]...okay!
  4. Check for Qt classes that should not be used [qclasses]...OOPS! 14 issues found!
     • plugins/scripting/ScriptingDataQueryView.ui: QComboBox[KComboBox]
     • libs/ui/kptaccountsviewconfigurepanelbase.ui: QComboBox[KComboBox],QComboBox[KComboBox]
     • libs/ui/kpttaskstatusviewsettingspanel.ui: QComboBox[KComboBox]
     • libs/ui/kptwbsdefinitionpanelbase.ui: QLineEdit[KLineEdit],QLineEdit[KLineEdit],QComboBox[KComboBox],QLineEdit[KLineEdit]
     • libs/ui/locale/localemon.ui: QLineEdit[KLineEdit],QComboBox[KComboBox],QComboBox[KComboBox]
     • libs/ui/kptperformancestatus.ui: QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit]
     • libs/ui/kpttaskcostpanelbase.ui: QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox]
     • libs/ui/kpttaskprogresspanelbase.ui: QTabWidget[KTabWidget],QComboBox[KComboBox]
     • libs/ui/resourcedialogbase.ui: QLineEdit[KLineEdit],QLineEdit[KLineEdit],QLineEdit[KLineEdit],QComboBox[KComboBox],QComboBox[KComboBox],QComboBox[KComboBox],QLineEdit[KLineEdit],QLineEdit[KLineEdit]
     • kptconfigtaskpanelbase.ui: QTabWidget[KTabWidget]
     • kptviewlisteditview.ui: QLineEdit[KLineEdit],QLineEdit[KLineEdit]
     • kptviewlistdocker.ui: QComboBox[KComboBox],QLineEdit[KLineEdit]
     • kptviewlisteditcategory.ui: QLineEdit[KLineEdit],QLineEdit[KLineEdit]
     • kptviewlistaddview.ui: QLineEdit[KLineEdit],QLineEdit[KLineEdit]

     Don't use Qt 4 classes that are deprecated. Additionally make sure to use the KDE version of some Qt GUI elements to provide a consistent look and feel for the KDE desktop. The KDE classes are not just adding functionalities to the Qt base class and are mostly not even based on the Qt class. http://techbase.kde.org/Policies/API_to_Avoid Please refer to the API documentation for details before porting to the K classes.

  5. Check for spelling errors [spelling]...okay!
• For File Type kconfigxt
  1. Check that file ends with a newline [endswithnewline]...okay!
  2. Check validity of i18n calls [i18ncheckarg]...okay!
  3. Check for spelling errors [spelling]...okay!
  4. Validates KConfigXT files using 'xmllint' [validate]...OOPS! 1 issues found!
     • kplatosettings.kcfg: 7

     Please make sure your .kcfg files conform to the KConfigXT schema