Home >  Krazy Code Checker >  kde-4.x >  kdeutils >  okteta

krazy2 Analysis

Checkers Run = 49
Files Processed = 1051
Total Issues = 9 ...as of March 17 2010 05:29:16 CET

• For File Type c++
  1. Check for TRUE and FALSE macros [captruefalse]...okay!
  2. Check for methods that return 'const' refs in public classes [constref]...okay!
  3. Check for an acceptable copyright [copyright]...okay!
  4. Check for cpp macros and usage [cpp]...OOPS! 2 issues found!
     • gui/offsetformat.cpp: O/S or Compiler specific macro line# 28[[__GNUC__]] (1)
     • gui/kcursor.cpp: O/S or Compiler specific macro line# 25[[__GNUC__]] (1)

     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.

  5. Check for code that should be considered crashy. [crashy]...OOPS! 5 issues found!
     • libs/kasten/controllers/documentsystem/creator/creatorcontroller.cpp: line# 143 (1)
     • libs/kasten/controllers/io/export/exportcontroller.cpp: line# 127 (1)
     • libs/kasten/controllers/io/copyas/copyascontroller.cpp: line# 133 (1)
     • libs/kasten/controllers/io/insert/insertcontroller.cpp: line# 130 (1)
     • libs/kasten/core/system/modelcodecmanager.cpp: line# 135 (1)

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

  6. Check single-char QString operations for efficiency [doublequote_chars]...okay!
  7. Check public classes with private members or d-pointer issues [dpointer]...okay!
  8. Check for QString compares to "" [emptystrcompare]...okay!
  9. Check that file ends with a newline [endswithnewline]...okay!
  10. Check for C++ ctors that should be declared 'explicit' [explicit]...okay!
  11. Check for foreach loop issues [foreach]...okay!
  12. Check validity of i18n calls [i18ncheckarg]...OOPS! 1 issues found!
      • kasten/controllers/view/structures/datatypes/datainformation.cpp: gaps in placeholder numbering, () (1 2) line# 65 (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.

  13. Check for invalid icon names [iconnames]...okay!
  14. Check for proper include directives [includes]...okay!
  15. Check for inline methods in public classes [inline]...okay!
  16. Check for an acceptable license [license]...okay!
  17. Check for assignments to QString::null [nullstrassign]...okay!
  18. Check for compares to QString::null or QString() [nullstrcompare]...okay!
  19. Check for C++ operators that should be 'const' [operators]...okay!
  20. Check for inappropriate pass-by-value function args [passbyvalue]...okay!
  21. Check for postfix usage of ++ and -- [postfixop]...okay!
  22. Check for dangerous or inefficient QByteArray usage [qbytearray]...okay!
  23. Check for Qt classes that should not be used [qclasses]...okay!
  24. Check for Qt methods that should be avoided [qmethods]...okay!
  25. Check for QMIN and QMAX macros [qminmax]...okay!
  26. Check for classes that should use the 'Q_OBJECT' macro [qobject]...okay!
  27. Check for signals: and slots: [sigsandslots]...okay!
  28. Check for spelling errors [spelling]...okay!
  29. Check for strings used improperly or should be i18n. [strings]...okay!
  30. Check for system calls to replace by KDE or Qt equivalents [syscalls]...OOPS! 1 issues found!
      • kasten/controllers/document/info/bytearraymodeliodevice.cpp: line# 37[open[KDE_open]] (1)

      Some system calls are not portable, please use the suggested portable wrapper instead. See kde_file.h

  31. Check for typedefs that should be replaced by Qt typedefs [typedefs]...okay!
• For File Type messages
  1. Check for appending to rc.cpp [rcappend]...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 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 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]...okay!
  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]...okay!