Commit 80379a37 authored by Jan Möbius's avatar Jan Möbius

Fixed a lot of cppcheck warnings

git-svn-id: http://www.openflipper.org/svnrepo/OpenFlipper/branches/Free@13304 383ad7c9-94d9-4d36-a494-682f7c89f535
parent bfa71c36
......@@ -64,10 +64,25 @@
#define TYPETRIANGLE 3
/// Constructor
FileOMPlugin::FileOMPlugin()
: loadOptions_(0),
saveOptions_(0),
triMeshHandling_(0) {
FileOMPlugin::FileOMPlugin() :
loadOptions_(0),
saveOptions_(0),
saveBinary_(0),
saveVertexNormal_(0),
saveVertexTexCoord_(0),
saveVertexColor_(0),
saveFaceColor_(0),
saveFaceNormal_(0),
saveDefaultButton_(0),
triMeshHandling_(0),
loadVertexNormal_(0),
loadVertexTexCoord_(0),
loadVertexColor_(0),
loadFaceColor_(0),
loadFaceNormal_(0),
loadDefaultButton_(0)
{
}
//-----------------------------------------------------------------------------------------------------
......@@ -589,4 +604,4 @@ void FileOMPlugin::slotSaveDefault() {
OpenFlipperSettings().setValue( "FileOM/Save/FaceNormal", saveFaceNormal_->isChecked() );
}
Q_EXPORT_PLUGIN2( fileomplugin , FileOMPlugin );
\ No newline at end of file
Q_EXPORT_PLUGIN2( fileomplugin , FileOMPlugin );
This diff is collapsed.
......@@ -104,7 +104,7 @@ private slots:
public:
// standard constructor/destructor
FilePTSPlugin() : loadOptions_( 0 ), saveOptions_( 0 ) { }
FilePTSPlugin();
~FilePTSPlugin() { }
//-- Base Interface --
......
......@@ -113,8 +113,11 @@ int FilePlaPlugin::loadObject(QString _filename)
plane->setFromFileName(_filename);
}
}
emit updatedObject( plane->id() );
}
emit updatedObject( plane->id() );
}
return id;
......
......@@ -75,22 +75,40 @@
*
*/
MovePlugin::MovePlugin() :
placeAndSnapAction_(0),
manMode_(QtTranslationManipulatorNode::TranslationRotation),
contextAction_(0),
toAllTargets_(0),
placeMode_(false)
axisA_(0),
axisB_(1),
toolboxActive_(false),
tool_(0),
toolIcon_(0),
moveAction_(0),
moveSelectionAction_(0),
toolBarActions_(0),
toolbar_(0),
pickToolbar_(0),
placeAction_(0),
rotateTranslateAction_(0),
rotateManipAction_(0),
resizeAction_(0),
biggerManipAction_(0),
smallerManipAction_(0),
fixChildManipAction_(0),
transformRefPoseManipAction_(0),
currentPoseManipAction_(0),
placeAndSnapAction_(0),
pickToolBarActions_(0),
manip_size_(1.0),
manip_size_modifier_(1.0),
lastActiveManipulator_(-1),
manMode_(QtTranslationManipulatorNode::TranslationRotation),
selectionType_(VERTEX),
contextAction_(0),
contextActionHide_(0),
toAllTargets_(0),
hide_(true),
allTargets_(false),
placeMode_(false)
{
manip_size_ = 1.0;
manip_size_modifier_ = 1.0;
selectionType_ = VERTEX;
axisA_ = 0;
axisB_ = 1;
hide_ = true;
allTargets_ = false;
}
/** \brief Deconstructor
......
......@@ -228,7 +228,7 @@ void SkeletalAnimationPlugin::checkObjectSelection(){
activeSkeletons_.push_back( o_it->id() );
// if no target skeleton there check if there is only one skeleton
if ( activeSkeletons_.size() == 0){
if ( activeSkeletons_.empty() ){
for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, DataType(DATA_SKELETON)) ; o_it != PluginFunctions::objectsEnd(); ++o_it)
activeSkeletons_.push_back( o_it->id() );
......@@ -240,7 +240,7 @@ void SkeletalAnimationPlugin::checkObjectSelection(){
activeMeshes_.push_back( o_it->id() );
// if no target mesh there check if there is only one mesh
if ( activeMeshes_.size() == 0){
if ( activeMeshes_.empty() ){
for ( PluginFunctions::ObjectIterator o_it(PluginFunctions::ALL_OBJECTS, DataType(DATA_TRIANGLE_MESH|DATA_POLY_MESH)) ; o_it != PluginFunctions::objectsEnd(); ++o_it)
activeMeshes_.push_back( o_it->id() );
......@@ -433,7 +433,7 @@ void SkeletalAnimationPlugin::slotAnimationIndexChanged(int /*_index*/)
if(bGuiUpdating_) // do not update while the gui is updating
return;
if ( activeSkeletons_.size() == 0){
if ( activeSkeletons_.empty() ){
pToolbox_->hsFrame->setRange( 0, 0 );
pToolbox_->hsFrame->setTickInterval(1);
pToolbox_->hsFrame->setValue(0);
......
......@@ -13,20 +13,35 @@
/** \brief Default Constructor
*
*/
SkeletonEditingPlugin::SkeletonEditingPlugin()
: currentSkeleton_(-1),
currentJoint_(-1),
jointPreview_(false),
transformChildJoints_(false),
transformAllFrames_(true),
inverseKinematic_(false),
dblClick_(false),
lastRenderer_(""),
rendererChanged_(false)
SkeletonEditingPlugin::SkeletonEditingPlugin() :
toolbar_(0),
toolBarActions_(0),
skeletonEditingAction_(0),
pickToolbar_(0),
pickToolBarActions_(0),
selectJointAction_(0),
insertJointAction_(0),
splitJointAction_(0),
deleteJointAction_(0),
moveJointAction_(0),
transformChildManipAction_(0),
transformAllManipAction_(0),
rotateManipAction_(0),
inverseKinematicAction_(0),
currentSkeleton_(-1),
currentJoint_(-1),
jointPreview_(false),
transformChildJoints_(false),
transformAllFrames_(true),
inverseKinematic_(false),
dblClick_(false),
lastRenderer_(""),
rendererChanged_(false),
manip_size_(1.0),
manip_size_modifier_(1.0)
{
manip_size_ = 1.0;
manip_size_modifier_ = 1.0;
}
//--------------------------------------------------------------------------------
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment