| [qet] Rapports cppcheck pour QElectroTech |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/qet Archives
]
- To: qet@xxxxxxxxxxxxxxxxxxx
- Subject: [qet] Rapports cppcheck pour QElectroTech
- From: Julien Nabet <serval2412@xxxxxxxx>
- Date: Sat, 05 Oct 2013 06:07:00 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.fr; s=s1024; t=1380946021; bh=hIRhmAQd/x3KiGgOG6CosZ2FzD1q3W20Snhfmj7EsFM=; h=X-Yahoo-Newman-Id:X-Yahoo-Newman-Property:X-YMail-OSG:X-Yahoo-SMTP:X-Rocket-Received:Message-ID:Date:From:User-Agent:MIME-Version:To:Subject:Content-Type; b=tSDDirla2NOJoPnM+6VCxV/EAIM7uipYUYSf6o55GIrd5kXyz/txtwjd3CCWx824/5An2LZ1Mk8NtsstkfDmNQbyEONGSG1dKzzeR8hOVu3ZtvcE+0L8iEgaHkdbvI2KMk2ZQR+kwkdhBamJFAj3JEcJpL7hqXBldSQj/CMDKB8=
Bonjour,
Si cela peut aider, j'ai lancé cppcheck (un outil libre et gratuit
d'analyse statique pour C++) sur QElectroTech, vous trouverez ci-joint
les rapports.
Julien Nabet
[sources/diagram.h:203]: (style) C-style pointer casting
[sources/conductorprofile.cpp:30]: (warning) Member variable 'ConductorProfile::beginOrientation' is not initialized in the constructor.
[sources/conductorprofile.cpp:30]: (warning) Member variable 'ConductorProfile::endOrientation' is not initialized in the constructor.
[sources/diagram.cpp:612]: (style) C-style pointer casting
[sources/diagramprintdialog.cpp:356]: (performance) Function parameter 'options' should be passed by reference.
[sources/diagramprintdialog.cpp:502]: (performance) Function parameter 'options' should be passed by reference.
[sources/diagramschooser.cpp:116]: (style) The scope of the variable 'check_box' can be reduced.
[sources/integrationmoveelementshandler.h:59] -> [sources/basicmoveelementshandler.h:60]: (warning) The class 'IntegrationMoveElementsHandler' defines member variable with name 'rename_' also defined in its parent class 'BasicMoveElementsHandler'.
[sources/editor/elementscene.cpp:245]: (style) The scope of the variable 'terminal' can be reduced.
[sources/editor/elementscene.cpp:246]: (style) The scope of the variable 'text' can be reduced.
[sources/editor/elementscene.cpp:247]: (style) The scope of the variable 'textfield' can be reduced.
[sources/editor/parttext.cpp:419]: (style) Consecutive return, break, continue, goto or throw statements are unnecessary.
[sources/editor/parttextfield.cpp:177]: (style) Consecutive return, break, continue, goto or throw statements are unnecessary.
[sources/interactivemoveelementshandler.h:73] -> [sources/basicmoveelementshandler.h:60]: (warning) The class 'InteractiveMoveElementsHandler' defines member variable with name 'rename_' also defined in its parent class 'BasicMoveElementsHandler'.
[sources/titleblock/qettemplateeditor.cpp:692]: (error) Uninitialized variable: can_merge
[sources/titleblock/qettemplateeditor.cpp:693]: (error) Uninitialized variable: can_split
[sources/titleblock/qettemplateeditor.cpp:687]: (error) Uninitialized variable: count
[sources/ui/diagramselection.cpp:103]: (style) Variable 'sel' is assigned a value that is never used.
[sources/xmlelementdefinition.h:66] -> [sources/elementdefinition.h:125]: (warning) The class 'XmlElementDefinition' defines member variable with name 'parent_category_' also defined in its parent class 'ElementDefinition'.